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

Allow the SOAP envelope header to be set as a String #289

Closed
wants to merge 1 commit into from

Conversation

Empact
Copy link
Contributor

@Empact Empact commented May 14, 2012

Requiring Gyoku for the header adds a layer of unnecessary indirection and overhead, particularly when the data is more natural in its native XML, e.g. when many attributes are involved.

@Empact
Copy link
Contributor Author

Empact commented May 14, 2012

See the succinctness of the header string setter in the spec for a case in point.

@JimPanic
Copy link

+1!

@rubiii
Copy link
Contributor

rubiii commented May 17, 2012

thanks! merged and scheduled for v0.9.10.

@rubiii rubiii closed this May 17, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants