Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-534

Request JMS correlation id field not used to set response message JMS correlation id

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.5.4
    • Fix Version/s: 1.5.8
    • Component/s: None
    • Labels:
      None
    • Environment:
      Windows XP SP2

      Description

      In JmsReceiverConnection.onSendBeforeWrite(...) the following code sets the response correlation id field :-

      >> responseMessage.setJMSCorrelationID(requestMessage .getJMSMessageID());

      There does not seem to be a check on the inbound request's correlation id as in the following code from JmsInvokerServiceExporter.createResponseMessage(.. .)

      >>String correlation = request.getJMSCorrelationID();
      >>if (correlation == null)

      { >>correlation = request.getJMSMessageID(); >>}

      >>response.setJMSCorrelationID(correlation);

      The above logic has also been implemented in MessageListnenerAdapter and I am looking for the same behaviour in JmsReceiverConnection.onSendBeforeWrite(...).

      This seems to be related to http://jira.springframework.org/browse/SPR-4647

        Activity

        gflood Gary Flood created issue -
        arjen.poutsma Arjen Poutsma made changes -
        Field Original Value New Value
        Fix Version/s 1.5.8 [ 11236 ]
        Issue Type Bug [ 1 ] Improvement [ 4 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        arjen.poutsma Arjen Poutsma made changes -
        Time Spent 7m [ 420 ]
        Remaining Estimate 0d [ 0 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            gflood Gary Flood
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0d
              0d
              Logged:
              Time Spent - 7m
              7m