Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.8.4, 2.9.0
    • Component/s: camel-cxf
    • Labels:
      None
    • Patch Info:
      Patch Available
    • Estimated Complexity:
      Unknown

      Description

      If you declare an endpoint like
      <cxf:rsServer id="testEndpoint" address="address" serviceClass="org.apache.camel.component.cxf.jaxrs.testbean.CustomerService" />
      the address is not resolve with the properties.

      It would be useful, as it is possible for CxfEndpoint.

      You can find with this issue, a small patch with a unit test to reproduce this use case, and a (naive but working) patch that fix the issue for us.

      1. cxfrs-properties-test.diff
        3 kB
        Cédric Exbrayat
      2. cxfrs-properties.diff
        3 kB
        Cédric Exbrayat
      3. cxfrs-properties-404-test.diff
        2 kB
        Cédric Exbrayat
      4. cxfrs-properties-404.diff
        2 kB
        Cédric Exbrayat

        Activity

        Cédric Exbrayat created issue -
        Cédric Exbrayat made changes -
        Field Original Value New Value
        Attachment cxfrs-properties-test.diff [ 12507701 ]
        Cédric Exbrayat made changes -
        Attachment cxfrs-properties.diff [ 12507702 ]
        Willem Jiang made changes -
        Assignee Willem Jiang [ njiang ]
        Hide
        Willem Jiang added a comment -

        Applied the part of patch with thanks to Cedric.
        I changed the part of how to get the address from the CxfRsEndpoint and also add the spring configuration file of testing which is missed from the patch.

        Show
        Willem Jiang added a comment - Applied the part of patch with thanks to Cedric. I changed the part of how to get the address from the CxfRsEndpoint and also add the spring configuration file of testing which is missed from the patch.
        Willem Jiang made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.9.0 [ 12316374 ]
        Resolution Fixed [ 1 ]
        Hide
        Cédric Exbrayat added a comment -

        Willem, the actual version does not work, I think that the address should be also modified in the AbstractJAXRSFactoryBean (as I was trying to do in the first rough patch).

        You can find a patch with an enhanced unit test that just check if the connection is ok (even with 404). Actually this test failed.
        The other patch just show how it can be fixed to pass the test

        Show
        Cédric Exbrayat added a comment - Willem, the actual version does not work, I think that the address should be also modified in the AbstractJAXRSFactoryBean (as I was trying to do in the first rough patch). You can find a patch with an enhanced unit test that just check if the connection is ok (even with 404). Actually this test failed. The other patch just show how it can be fixed to pass the test
        Cédric Exbrayat made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Cédric Exbrayat made changes -
        Attachment cxfrs-properties-404-test.diff [ 12508075 ]
        Cédric Exbrayat made changes -
        Attachment cxfrs-properties-404.diff [ 12508076 ]
        Hide
        Willem Jiang added a comment -

        Sorry we just missed the release train of Camel 2.9.0.

        Show
        Willem Jiang added a comment - Sorry we just missed the release train of Camel 2.9.0.
        Willem Jiang made changes -
        Fix Version/s 2.9.1 [ 12319191 ]
        Fix Version/s 2.9.0 [ 12316374 ]
        Affects Version/s 2.9.0 [ 12316374 ]
        Affects Version/s 2.10 [ 12317612 ]
        Hide
        Willem Jiang added a comment -

        Applied patch into trunk with thanks to Cédric.

        Show
        Willem Jiang added a comment - Applied patch into trunk with thanks to Cédric.
        Willem Jiang made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Daniel Kulp made changes -
        Fix Version/s 2.8.4 [ 12319072 ]
        Fix Version/s 2.9.0 [ 12316374 ]
        Fix Version/s 2.9.1 [ 12319191 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        23h 52m 1 Willem Jiang 17/Dec/11 14:56
        Resolved Resolved Reopened Reopened
        2d 22h 58m 1 Cédric Exbrayat 20/Dec/11 13:55
        Reopened Reopened Resolved Resolved
        14h 34m 1 Willem Jiang 21/Dec/11 04:30

          People

          • Assignee:
            Willem Jiang
            Reporter:
            Cédric Exbrayat
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development