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

Fix a problem when http.body is empty #800

Merged
merged 2 commits into from
Jul 29, 2017
Merged

Conversation

syguer
Copy link
Contributor

@syguer syguer commented Mar 13, 2017

When http.body is empty, SOAPFault#to_s raises error because SOAPFault#to_hash called by SOAPFault#to_s returns nil.

I think #to_hash should returns Hash rather than nil 🤔

Izumiya Keisuke added 2 commits March 13, 2017 17:25
When http.body is empty, #to_s raises error.
Because #to_hash called by #to_s returns nil
@syguer
Copy link
Contributor Author

syguer commented Mar 13, 2017

spec will be all green after #799 is merged

@syguer
Copy link
Contributor Author

syguer commented Mar 13, 2017

Should I bump up version?

@pcai pcai merged commit cd7efa1 into savonrb:master Jul 29, 2017
@pcai
Copy link
Member

pcai commented Jul 29, 2017

Thanks for the fix!

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.

2 participants