JimmyAlegre / ksoap2-android

Automatically exported from code.google.com/p/ksoap2-android
0 stars 0 forks source link

SOAPActionHeader should not be included when using Soap v. 1.2 #67

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Try calling a (conformant) Soap12 service
2. Even if you .call() with SoapAction as null the SoapAction header field will 
be set. 
3. This is not valid for a Soap 1.2 service, so the call will fail. 

The bug is at line 122 in HttpTransportSE.java, since there is no checking for 
soap version. 

Proposed fix: 

if (envelope.version == SoapSerializationEnvelope.VER12) {
 // do not add SoapAction header 
} else {
  connection.setRequestProperty("SOAPAction", soapAction);
}

Original issue reported on code.google.com by pet...@everichon.com on 6 Jul 2011 at 7:22

GoogleCodeExporter commented 9 years ago
Fix applied. Just waiting for author details and will release after I got the 
details.

https://github.com/mosabua/ksoap2-android/commit/d5752cf68aaea135068d7a7db06cd62
c33a917ce

Original comment by mosa...@gmail.com on 6 Jul 2011 at 8:37

GoogleCodeExporter commented 9 years ago
will be out with 2.5.7

Original comment by mosa...@gmail.com on 6 Jul 2011 at 10:57