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

Line breaks added in the body content #153

Closed
jlecour opened this issue Apr 5, 2013 · 3 comments
Closed

Line breaks added in the body content #153

jlecour opened this issue Apr 5, 2013 · 3 comments

Comments

@jlecour
Copy link

jlecour commented Apr 5, 2013

Hi,

I was having issues with a SOAP client I'm writing, based on Savon, Nori, Nokogiri, HTTPi and restclient.

I've had xml parsing errors, because of line breaks in the middle of a couple of xml tags. I first reproduced the request in a bare curl invocation, to confirm that there is no line break in the response from the server.

I've tried to debug my way into restclient to find where the raw http response as processed, but I got lost and decided that 1 hour spent on this was a little too much.

I've swapped the http client from httpclient to curb and retried ; it worked.

The deepest I got is in HTTPClient#do_request() where the res variable at the end contains the XML string, with the line breaks.
I couldn't manage to go deeper and find where those line breaks com from.

Sadly, the SOAP request is not public.
The response is 38027 characters long. Line breaks are at position 16377, then 16381.

@lukebakken
Copy link

@jlecour - what version of Ruby are you using? #161 could possibly be related.

@phoet
Copy link

phoet commented May 24, 2013

this, or a similar problem, occurs to me.

i am switching to ruby 2.0.0 builds and HTTPClient 2.3.3 seems to have problems with the Picasa endpoint that i am fetching data from.

i created a gist with the output of the call for the two ruby versions: https://gist.github.com/phoet/5645555

@nahi
Copy link
Owner

nahi commented Oct 19, 2014

Fixed this issue by different refactoring eventually.

@nahi nahi closed this as completed Oct 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants