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

Transfer-Encoding: chunked #6

Open
billmse opened this issue Mar 8, 2013 · 2 comments
Open

Transfer-Encoding: chunked #6

billmse opened this issue Mar 8, 2013 · 2 comments

Comments

@billmse
Copy link

billmse commented Mar 8, 2013

I think this library should add some codes to support Transfer-Encoding: chunked.

@foosel
Copy link

foosel commented Jun 16, 2013

Or alternatively not state that it speaks HTTP/1.1 (which makes it mandatory to be able to cope with chunked encoding) but instead be honest and send out a HTTP/1.0

@DefProc
Copy link

DefProc commented Jun 12, 2015

While processing additional headers may be more difficult checking content length from chunked encoding and removing the chunk length numbers would be slightly easier to include.

I came up against this talking to an api. Content from an api that set Transfer-Encoding: chunked would return:


f
{"nonce":11111}
0

in the serial monitor. f\r\n indicates the 15 character response, and 0\r\n indicates the end of the request.

This library should at least strip the chunk lengths trailing CRLF, and ideally the spurious CRLF after each chunk.

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

3 participants