Hi Gilbert,

I am sorry that my previous reply was not true.
I remembered when we first wrote Sandesha we added that redundant envelope creation to get it working. I can't remember why though. Somehow this old code segment has carried forward and now that requirement is not there and it is working without the redundant SOAP envelope creation.

I have made the change and committed the code. Please take a new checkout.

Thanks,
-Jaliya

----- Original Message ----- From: "Gilbert Pilz" <[EMAIL PROTECTED]>
To: "Jaliya Ekanayake" <[EMAIL PROTECTED]>; <[email protected]>
Sent: Friday, September 15, 2006 4:57 PM
Subject: RE: Sandesha1 code question


A work-around for what precisely? I'm writing a new method for
RMMessageCreator that creates an RMMessageContext that will be used to
send a SequenceFault. How can I tell whether I should employ the same
work-around?

- gp

-----Original Message-----
From: Jaliya Ekanayake [mailto:[EMAIL PROTECTED]
Sent: Friday, September 15, 2006 12:10 PM
To: Gilbert Pilz; [email protected]
Subject: Re: Sandesha1 code question

Yep, it was a work around. If you could find the solution
please send us a patch.
Thanks,
-Jaliya


----- Original Message ----- From: "Gilbert Pilz" <[EMAIL PROTECTED]>
To: <[email protected]>
Sent: Friday, September 15, 2006 1:00 PM
Subject: Sandesha1 code question


I have a question about the Sandesha1 code. In
org.apache.sandesha.util.RMMessageCreator.createTerminateSequence()
there are the following lines of code:

(1) AddressingHeaders addHeaders =
getAddressingHeadersForTerminateSequence(rmMsgCtx, endPoint);
(2) SOAPEnvelope soe = new SOAPEnvelope();
(3) addHeaders.toEnvelope(soe);
(4) AddressingHeaders terSqAddrHeaders = new AddressingHeaders(soe);
(5) terSqAddrHeaders.setAction(new Action(new
URI(Constants.WSRM.ACTION_TERMINATE_SEQUENCE)));
(6) terSeqRMMsgContext.setAddressingHeaders(terSqAddrHeaders);

So (1), obviously, is a call to an internal method to construct the
WS-Addressing headers needed for a TerminateSequence message.
Its (2-4)
that are puzzling me. Why create a SOAPEnvelope (that is never used
again BTW), add the headers to this envelope, then turn around and
construct another set of headers from this envelope? Is this some kind
of work-around to a bug in either Axis or Apache Addressing?

- gp

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to