You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tuscany.apache.org by "ant elder (JIRA)" <de...@tuscany.apache.org> on 2009/01/30 16:31:02 UTC

[jira] Commented: (TUSCANY-2798) Response destination not used in when JMSReplyTo is absent

    [ https://issues.apache.org/jira/browse/TUSCANY-2798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12668900#action_12668900 ] 

ant elder commented on TUSCANY-2798:
------------------------------------

I've committed a fix for this in r739306 and an itest to verify in r739304, please try and see if this resolves the problem.
Slightly rushed so i'll leave the JIRA open and review it again on Monday.

> Response destination not used in <binding.jms> when JMSReplyTo is absent
> ------------------------------------------------------------------------
>
>                 Key: TUSCANY-2798
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-2798
>             Project: Tuscany
>          Issue Type: Bug
>          Components: Java SCA JMS Binding Extension
>            Reporter: Scott Kurz
>            Assignee: ant elder
>
> The OASIS JMS Binding spec makes this clear whereas the OSOA doesn't, but if the JMSReplyTo isn't present on the incoming requestMessage then the response destination should be used.
> Not sure the best place to tweak the code to add this just yet, but you can see in TransportServiceInterceptor what we do today in sca-java-1.x
>     public Message invokeResponse(Message msg) { 
>      ...
>             if (requestJMSMsg.getJMSReplyTo() == null) {
>                 // assume no reply is expected
>                 if (msg.getBody() != null) {
>                     logger.log(Level.FINE, "JMS service '" + service.getName() + "' dropped response as request has no replyTo");
>                 }
>                 return msg;
>             }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.