Skip to content

fix: require Host on HTTP/1.1 requests#162

Open
anmonteiro wants to merge 1 commit intoanmonteiro/h1spec-missing-http-versionfrom
anmonteiro/h1spec-missing-host
Open

fix: require Host on HTTP/1.1 requests#162
anmonteiro wants to merge 1 commit intoanmonteiro/h1spec-missing-http-versionfrom
anmonteiro/h1spec-missing-host

Conversation

@anmonteiro
Copy link
Owner

Summary

  • reject HTTP/1.1 requests that omit the required Host header
  • validate request header structure before selecting the body reader
  • add a server-connection regression covering the h1spec missing-Host case

Testing

  • dune runtest --no-buffer
  • nix develop -c make h1spec

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

Successfully merging this pull request may close these issues.

1 participant