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

[http] add rfc7230 handling #451

Merged
merged 2 commits into from May 29, 2019

Conversation

2 participants
@zekth
Copy link
Contributor

commented May 24, 2019

Add handling of RFC 7230:
https://tools.ietf.org/html/rfc7230#section-3.3.2

      A sender MUST NOT send a Content-Length header field in any message
      that contains a Transfer-Encoding header field.

@zekth zekth referenced this pull request May 28, 2019

Closed

Http #461

@piscisaureus piscisaureus force-pushed the zekth:add_rfc7230 branch from a342461 to f041910 May 29, 2019

@piscisaureus piscisaureus self-requested a review May 29, 2019

@piscisaureus piscisaureus force-pushed the zekth:add_rfc7230 branch from f041910 to 130ccba May 29, 2019

@piscisaureus
Copy link
Contributor

left a comment

LGTM

@piscisaureus piscisaureus merged commit 1db594d into denoland:master May 29, 2019

5 checks passed

denoland.deno_std Build #20190529.26 succeeded
Details
denoland.deno_std (Linux) Linux succeeded
Details
denoland.deno_std (Mac) Mac succeeded
Details
denoland.deno_std (Windows) Windows succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@zekth zekth deleted the zekth:add_rfc7230 branch May 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.