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.

Sign up for GitHub

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

Jump to bottom

2.2 Message parsing: mention close after 400 #750

Closed
wtarreau opened this issue Feb 7, 2021 · 0 comments · Fixed by #759
Closed

2.2 Message parsing: mention close after 400 #750

wtarreau opened this issue Feb 7, 2021 · 0 comments · Fixed by #759
Labels
h1-messaging has-proposal

Comments

@wtarreau
Copy link

wtarreau commented Feb 7, 2021

After "the server SHOULD respond with a 400 (Bad Request) response" I'd add "and close the connection" as it's not necessarily obvious to everyone.
mnot added a commit that referenced this issue Feb 10, 2021
@mnot
Explicitly close the connection
b4e5514 
Fixes #750
@mnot mnot mentioned this issue Feb 10, 2021
Merged
reschke added a commit that referenced this issue Feb 17, 2021
@reschke
fix changelog (#750)
226b50d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
h1-messaging has-proposal
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Explicitly close the connection httpwg/http-core
3 participants
@mnot @reschke @wtarreau