You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by mm...@apache.org on 2007/12/21 08:07:48 UTC
svn commit: r606119 - in /incubator/cxf/trunk:
rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java
systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java
Author: mmao
Date: Thu Dec 20 23:07:44 2007
New Revision: 606119
URL: http://svn.apache.org/viewvc?rev=606119&view=rev
Log:
* SOAPAction in Http headers should be quoted anyway
Modified:
incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java
incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java
Modified: incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java?rev=606119&r1=606118&r2=606119&view=diff
==============================================================================
--- incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java (original)
+++ incubator/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionOutInterceptor.java Thu Dec 20 23:07:44 2007
@@ -92,10 +92,11 @@
} else {
SoapOperationInfo soi = (SoapOperationInfo) boi.getExtensor(SoapOperationInfo.class);
action = soi == null ? "\"\"" : soi.getAction() == null ? "\"\"" : soi.getAction();
- if (!action.startsWith("\"")) {
- action = new StringBuffer().append("\"").append(action).append("\"").toString();
- }
}
+ }
+
+ if (!action.startsWith("\"")) {
+ action = new StringBuffer().append("\"").append(action).append("\"").toString();
}
return action;
Modified: incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java?rev=606119&r1=606118&r2=606119&view=diff
==============================================================================
--- incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java (original)
+++ incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/addr_fromjava/WSAFromJavaTest.java Thu Dec 20 23:07:44 2007
@@ -60,7 +60,7 @@
String expectedOut = "http://cxf.apache.org/input";
assertTrue(output.toString().indexOf(expectedOut) != -1);
- assertTrue(output.toString().indexOf("SOAPAction=[cxf]") != -1);
+ assertTrue(output.toString().indexOf("SOAPAction=[\"cxf\"]") != -1);
String expectedIn = "http://cxf.apache.org/output";
assertTrue(input.toString().indexOf(expectedIn) != -1);