You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by ch...@apache.org on 2006/03/02 18:38:43 UTC
svn commit: r382457 -
/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Author: chinthaka
Date: Thu Mar 2 09:38:42 2006
New Revision: 382457
URL: http://svn.apache.org/viewcvs?rev=382457&view=rev
Log:
Fault must go to replyTo if faultTo is absent
Modified:
webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Modified: webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java?rev=382457&r1=382456&r2=382457&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java (original)
+++ webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java Thu Mar 2 09:38:42 2006
@@ -177,13 +177,14 @@
EndpointReference faultTo = processingContext.getFaultTo();
if (faultTo != null) {
faultContext.setTo(processingContext.getFaultTo());
- } else
- if (processingContext.getEnvelope().getHeader() != null && processingContext.getEnvelope().getHeader().getFirstChildWithName(new QName("FaultTo")) != null)
+ } else if (processingContext.getEnvelope().getHeader() != null && processingContext.getEnvelope().getHeader().getFirstChildWithName(new QName("FaultTo")) != null)
{
OMElement faultToElement = processingContext.getEnvelope().getHeader().getFirstChildWithName(new QName("FaultTo"));
faultTo = new EndpointReference("");
faultTo.fromOM(faultToElement);
faultContext.setTo(faultTo);
+ } else if (processingContext.getReplyTo() != null) {
+ faultContext.setTo(processingContext.getReplyTo());
}
if (faultTo == null || AddressingConstants.Final.WSA_ANONYMOUS_URL.equals(faultTo.getAddress())
Re: svn commit: r382457 - /webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Posted by Eran Chinthaka <ch...@opensource.lk>.
Yeah, agreed. I knew this will come up from someone. Let me fix this tomoro.
-- Chinthaka
Sanjiva Weerawarana wrote:
>On Thu, 2006-03-02 at 17:38 +0000, chinthaka@apache.org wrote:
>
>
>> faultTo = new EndpointReference("");
>> faultTo.fromOM(faultToElement);
>>
>>
>
>Not for right now (in the midst of interop testing ;-)) but we should
>change that to be a static method on EndpointReference:
>
>public static EndpointReference create (OMElement);
>
>Sanjiva.
>
>
>
>
Re: svn commit: r382457 - /webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Posted by Eran Chinthaka <ch...@opensource.lk>.
Yeah, agreed. I knew this will come up from someone. Let me fix this tomoro.
-- Chinthaka
Sanjiva Weerawarana wrote:
>On Thu, 2006-03-02 at 17:38 +0000, chinthaka@apache.org wrote:
>
>
>> faultTo = new EndpointReference("");
>> faultTo.fromOM(faultToElement);
>>
>>
>
>Not for right now (in the midst of interop testing ;-)) but we should
>change that to be a static method on EndpointReference:
>
>public static EndpointReference create (OMElement);
>
>Sanjiva.
>
>
>
>
Re: svn commit: r382457 -
/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Posted by Sanjiva Weerawarana <sa...@opensource.lk>.
On Thu, 2006-03-02 at 17:38 +0000, chinthaka@apache.org wrote:
>
> faultTo = new EndpointReference("");
> faultTo.fromOM(faultToElement);
Not for right now (in the midst of interop testing ;-)) but we should
change that to be a static method on EndpointReference:
public static EndpointReference create (OMElement);
Sanjiva.
Re: svn commit: r382457 -
/webservices/axis2/trunk/java/modules/core/src/org/apache/axis2/engine/AxisEngine.java
Posted by Sanjiva Weerawarana <sa...@opensource.lk>.
On Thu, 2006-03-02 at 17:38 +0000, chinthaka@apache.org wrote:
>
> faultTo = new EndpointReference("");
> faultTo.fromOM(faultToElement);
Not for right now (in the midst of interop testing ;-)) but we should
change that to be a static method on EndpointReference:
public static EndpointReference create (OMElement);
Sanjiva.