Skip to content

Unblock QUIC stream reads on request body close#1697

Open
cyphercodes wants to merge 1 commit into
cloudflare:masterfrom
cyphercodes:fix/quic-close-unblocks-read-1696
Open

Unblock QUIC stream reads on request body close#1697
cyphercodes wants to merge 1 commit into
cloudflare:masterfrom
cyphercodes:fix/quic-close-unblocks-read-1696

Conversation

@cyphercodes

Copy link
Copy Markdown

Summary

  • cancel the QUIC read side when nopCloserReadWriter.Close is called
  • keep the write side open so origin responses can still be forwarded
  • add a regression test for a pending read unblocked by request-body close

Fixes #1696

Tests

  • /root/go/bin/go1.26.5 test -mod=vendor ./connection -run 'TestNopCloserReadWriter|TestQUICServer'
  • /root/go/bin/go1.26.5 test -mod=vendor ./quic ./connection
  • /root/go/bin/go1.26.5 test -race -mod=vendor ./connection -run TestNopCloserReadWriterCloseUnblocksPendingRead
  • git diff --check

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.

🐛 QUIC request body Close does not unblock a pending Read, causing streaming requests to hang with http2Origin

1 participant