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

[mmao]

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);