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

Implement chunked TE #11

Open
3Hren opened this issue Jun 20, 2017 · 0 comments
Open

Implement chunked TE #11

3Hren opened this issue Jun 20, 2017 · 0 comments

Comments

@3Hren
Copy link
Owner

3Hren commented Jun 20, 2017

This can be tricky, because:

  • If a response is set Content-Length header - we should not encode the body.
  • If there neither Content-Length header nor Transfer-Encoding: chunked - we should encode the body. Hyper can do it easily.
  • If there is Transfer-Encoding: chunked - we should firstly decode the body and to encode it again to avoid double-encoding.

Another issue is - there is some kind of accept logging. In the case of chunked TE it will be broken, as Hyper provides no way to notify when the last body chunk was written.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant