You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tuscany.apache.org by "Simon Laws (Assigned) (JIRA)" <de...@tuscany.apache.org> on 2012/02/21 11:33:34 UTC

[jira] [Assigned] (TUSCANY-4018) JMS TransportServiceInterceptor using wrong values to set JMS headers in producer

     [ https://issues.apache.org/jira/browse/TUSCANY-4018?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Simon Laws reassigned TUSCANY-4018:
-----------------------------------

    Assignee: Simon Laws
    
> JMS TransportServiceInterceptor using wrong values to set JMS headers in producer
> ---------------------------------------------------------------------------------
>
>                 Key: TUSCANY-4018
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-4018
>             Project: Tuscany
>          Issue Type: Bug
>          Components: Java SCA JMS Binding Extension
>    Affects Versions: Java-SCA-2.x
>            Reporter: Jennifer A Thompson
>            Assignee: Simon Laws
>             Fix For: Java-SCA-2.x
>
>         Attachments: Tuscany-4018.patch
>
>
> In TransportServiceInterceptor.invokeResponse is setting the effective TimeToLive, Priority in the response message, but when the values are set in the producer, the values from the request are used, potentially leading to an incorrect value. So the following:
>              // Set jms header attributes in producer, not message.
>             int deliveryMode = requestJMSMsg.getJMSDeliveryMode();
>             producer.setDeliveryMode(deliveryMode);
>             int deliveryPriority = requestJMSMsg.getJMSPriority();
>             producer.setPriority(deliveryPriority);
>             long timeToLive = requestJMSMsg.getJMSExpiration();
>             producer.setTimeToLive(timeToLive);
> Should be:
>             // Set jms header attributes in producer, not message.
>             producer.setDeliveryMode(responseJMSMsg.getJMSDeliveryMode());
>             producer.setPriority(responseJMSMsg.getJMSPriority());
>             producer.setTimeToLive(responseJMSMsg.getJMSExpiration());

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira