[ http://issues.apache.org/jira/browse/SANDESHA2-55?page=all ]

Takahide Nogayama updated SANDESHA2-55:
---------------------------------------

    Attachment: patch3.txt

Hi,

The problem here is, some parameters were changed in 
TransportUtils.createSOAPMessage(MessageContext, String). and the change might 
be used in axisEngine or handler. So that TransportUtils , AxisEngine, and 
handlers share the same MessageContext, responseMessageContext is put to 
TransportUtils.createSOAPMessage(MessageContext, String).

This patch3.txt fix this problem and this patch passed all unit test.

# my environment is maven1.1-beta2 with Sun JDK 5.0 @ FC6 linux

Thanks.


> 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
>         Attachments: patch3.txt
>
>
> 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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to