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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 20h 4m 1 Krzysztof Sobkowiak 13/Jun/14 22:37
        Resolved Resolved Closed Closed
        290d 19h 53m 1 Krzysztof Sobkowiak 31/Mar/15 18:31
        Krzysztof Sobkowiak made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Krzysztof Sobkowiak made changes -
        Fix Version/s 6.0.0.M1 [ 12329337 ]
        Krzysztof Sobkowiak made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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!!!!
        Krzysztof Sobkowiak made changes -
        Fix Version/s 5.0.2 [ 12326849 ]
        Fix Version/s 5.1.0 [ 12326660 ]
        Fix Version/s 6.0.0 [ 12326661 ]
        Krzysztof Sobkowiak made changes -
        Field Original Value New Value
        Assignee Krzysztof Sobkowiak [ sobkowiak ]
        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
        Gregor Zurowski created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development