Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0.2
    • Fix Version/s: 5.0.2, 5.1.0, 6.0.0.M1, 6.0.0
    • Component/s: examples
    • Labels:
      None

      Description

      Address the following issues with the current camel-cxf-rest example implementation:

      • ServiceHandler should use a more appropriate data structure for storing person instances (Map instead of List).
      • ServiceHandler should follow REST principles by using appropriate HTTP response codes and messages (HTTP 201 when creating persons, return URI of created person, etc.).
      • ServiceHandler should follow JAX-RS programming model (use Response instance where appropriate).
      • Client should show HTTP response codes and response messages.
      • Fix some general issues (e.g. remove unneeded fields and annotations, use appropriate access modifiers)

        Activity

        Show
        Krzysztof Sobkowiak added a comment - Applied Gregors pull request in servicemix-5.1.x – https://git-wip-us.apache.org/repos/asf/servicemix/?p=servicemix.git;a=commit;h=c4dd2af3b1d59e1fb9a00587810390d3096f8b5b servicemix-5.0.x – https://git-wip-us.apache.org/repos/asf/servicemix/?p=servicemix.git;a=commit;h=034d43d4d043d0115d5e2351d0b9f146aea26f27 master – https://git-wip-us.apache.org/repos/asf/servicemix/?p=servicemix.git;a=commit;h=d164720e154352deaa5e7d9ef49e250e3798a767 Thanks to Gregor for the pull request!!!!
        Hide
        Gregor Zurowski added a comment -

        I have submitted the following PR for this ticket: https://github.com/apache/servicemix/pull/14

        Show
        Gregor Zurowski added a comment - I have submitted the following PR for this ticket: https://github.com/apache/servicemix/pull/14

          People

          • Assignee:
            Krzysztof Sobkowiak
            Reporter:
            Gregor Zurowski
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development