Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URI-reference is not quoted in Soapaction HTTP header #80

Closed
tindron opened this issue Aug 27, 2010 · 7 comments
Closed

URI-reference is not quoted in Soapaction HTTP header #80

tindron opened this issue Aug 27, 2010 · 7 comments

Comments

@tindron
Copy link

tindron commented Aug 27, 2010

Per section 6.1.1 of the SOAP 1.1 spec, the SOAPAction HTTP header must have double quotes surrounding the URI-reference (if present).

@alucardpj
Copy link

I also found this issue, I'm using savon with a ws published by JDK6, when calling the JDK6's ws using savon, it reports warning:

com.sun.xml.internal.ws.transport.http.HttpAdapter fixQuotesAroundSoapAction
Received WS-I BP non-conformant Unquoted SoapAction HTTP header: xxx

@rubiii
Copy link
Contributor

rubiii commented Oct 4, 2010

shouldn't the http header be handled by the http client? this looks like a simple problem to solve, but i would appreciate if you guys could point me in the right direction.

please note, that v0.8.0 uses the httpi gem.

@tindron
Copy link
Author

tindron commented Oct 4, 2010

Most HTTP headers aren't required to be quoted - the requirement is specific to the SOAPAction header. You'll need to include the quotes in the value you give to your HTTPI::Request.

@rubiii
Copy link
Contributor

rubiii commented Oct 4, 2010

thanks! let me check if this works with both HTTPClient and Curb.

@rubiii
Copy link
Contributor

rubiii commented Oct 6, 2010

hope this commit fixes your problem:
http://github.com/rubiii/savon/commit/635f30

@rubiii
Copy link
Contributor

rubiii commented Oct 29, 2010

please try savon 0.8.0.beta.1

@rubiii
Copy link
Contributor

rubiii commented Dec 20, 2010

released v0.8.0 which should solve this problem.

calamitas pushed a commit to calamitas/savon that referenced this issue Jun 7, 2011
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants