You are viewing a plain text version of this content. The canonical link for it is here.
Posted to sandesha-dev@ws.apache.org by "Matt Lovett (JIRA)" <ji...@apache.org> on 2006/11/30 17:08:23 UTC
[jira] Commented: (SANDESHA2-55) pass options to
responseMessageContext
[ http://issues.apache.org/jira/browse/SANDESHA2-55?page=comments#action_12454686 ]
Matt Lovett commented on SANDESHA2-55:
--------------------------------------
Unfortunately, copying over the options at this point seems to break the sandesha unit tests. From the look of the tcpmon output, we are failing to create the RM sequence. Perhaps you should try to come up with a better patch, contact me offline, or start a discussion on the mailing list.
Thanks, Matt
> pass options to responseMessageContext
> --------------------------------------
>
> Key: SANDESHA2-55
> URL: http://issues.apache.org/jira/browse/SANDESHA2-55
> Project: Apache Sandesha2
> Issue Type: Bug
> Reporter: Takahide Nogayama
> Assigned To: Matt Lovett
> Priority: Critical
>
> org.apache.sandesha2.workers.SenderWorker.checkForSyncResponses(MessageContext msgCtx) creates responseMessageContext and set some information from input argument messagecontext to responseMessageContext. I think that MessageContext.options should also be set to responseMessageContext. In current implementation, it is discarded.
> We can see similar case in org.apache.axis2.description.OutInAxisOperation.send(MessageContext msgctx). And the fix code is from here.
> -current------------------------------------------------------------------------------------------
> // create the responseMessageContext
> MessageContext responseMessageContext = new MessageContext();
> ------------------------------------------------------------------------------------------------------
> -fix-----------------------------------------------------------------------------------------------------
> // create the responseMessageContext
> MessageContext responseMessageContext = new MessageContext();
> responseMessageContext.setOptions(msgCtx.getOptions()); // set options.
> ------------------------------------------------------------------------------------------------------
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: sandesha-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: sandesha-dev-help@ws.apache.org