Skip to content

fix: reject request lines without an HTTP version#161

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

fix: reject request lines without an HTTP version#161
anmonteiro wants to merge 1 commit intoanmonteiro/h1spec-invalid-http-versionfrom
anmonteiro/h1spec-missing-http-version

Conversation

@anmonteiro
Copy link
Owner

Summary

  • make the request-line version slot fail immediately when it does not begin with HTTP/
  • keep fragmented request-line parsing intact while rejecting GET / \r\n\r\n
  • add a server-connection regression covering the h1spec missing-version case

Testing

  • dune runtest --no-buffer
  • PATH="/nix/store/a9d9dk3hbks4xb5p4lrn625ddkb7k6d0-h1spec/bin:$PATH" ./scripts/run-h1spec.sh

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