Skip to content

stream: track FIN acknowledgements separately - #2665

Open
xgame92 wants to merge 1 commit into
cloudflare:masterfrom
xgame92:fix/zero-length-stream-fin
Open

stream: track FIN acknowledgements separately#2665
xgame92 wants to merge 1 commit into
cloudflare:masterfrom
xgame92:fix/zero-length-stream-fin

Conversation

@xgame92

@xgame92 xgame92 commented Aug 26, 2026

Copy link
Copy Markdown

Summary

  • track FIN acknowledgement separately from acknowledged stream byte ranges
  • retain streams until a FIN-bearing STREAM frame is acknowledged
  • preserve reset and shutdown completion behavior
  • cover zero-length FIN delivery and retransmission with Cubic and BBR2

Testing

  • cargo +nightly fmt -- --check
  • cargo clippy --release -p quiche --all-targets -- -D warnings
  • cargo test --release --all-targets --features=async,ffi,qlog --workspace
  • cargo test --release --doc --features=async,ffi,qlog --workspace

Fixes #2525

@xgame92
xgame92 requested a review from a team as a code owner August 26, 2026 13:27
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.

zero-length STREAM FIN can be dropped before it is sent

1 participant