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

Encode message in encoding specified in globals #431

Closed
wants to merge 3 commits into from

Conversation

skorbut
Copy link

@skorbut skorbut commented Apr 12, 2013

Currently the encoding is only set on the request, but the message isn't actually converted. This leads to errors with german umlauts (For example in addressdata).

@rubiii
Copy link
Contributor

rubiii commented Apr 19, 2013

hey @skorbut,

although this looks to be "the right thing to do", it's a rather big change and i'm not sure if it just saves people from encoding the message themselves or if they actually need to do it manually.

@skorbut
Copy link
Author

skorbut commented Apr 19, 2013

Hi @rubiii,

I just looked for the best place to encode the whole soap request in one small step. We have quite big hashes, that I don't want to traverse recursively. Shall we add another global option to whether encode using the provided encoding?

@rubiii
Copy link
Contributor

rubiii commented Apr 19, 2013

a new option would certainly be an option. maybe something like :encode_message => true?

@rubiii rubiii mentioned this pull request Jul 24, 2013
27 tasks
@rubiii
Copy link
Contributor

rubiii commented Jul 26, 2013

i was planning to release this with v2.3.0. so i fixed the implementation and specs to work on 1.8 when i noticed there's also one failing spec for jruby. so i reverted it for now.

this needs to be updated against savon's version 2 branch and fixed to raise an error for 1.8 and jruby.

@skorbut
Copy link
Author

skorbut commented Aug 15, 2013

Hi,
this is an issue of jruby jruby/jruby#856 and is addressed in JRuby 1.7.5. Currently unrecognized characters are always replaced by a ? instead of the string provided to the encode method. I don't want to remove this spec, but may set it to pending until jruby 1.7.5 is used.

@stale
Copy link

stale bot commented Feb 19, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Feb 19, 2018
@stale
Copy link

stale bot commented Feb 26, 2018

This issue is now closed due to inactivity. If you believe this needs further action, please re-open to discuss.

@stale stale bot closed this Feb 26, 2018
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