Skip to content

fix: send MAX_STREAMS when available <= 50% of initial for long lived connections#2251

Merged
ghedo merged 5 commits intocloudflare:masterfrom
ehaydenr:flow-control
Feb 16, 2026
Merged

fix: send MAX_STREAMS when available <= 50% of initial for long lived connections#2251
ghedo merged 5 commits intocloudflare:masterfrom
ehaydenr:flow-control

Conversation

@ehaydenr
Copy link
Copy Markdown
Contributor

The current logic sends MAX_STREAMS too aggressively. As the next MAX_STREAMS value increases, the condition for whether or not to send MAX_STREAMS becomes unconditionally true. This change fixes that so that MAX_STREAMS is only sent when 50% or more streams have completed.

… connections

The current logic sends MAX_STREAMS too aggressively. As the next MAX_STREAMS value
increases, the condition for whether or not to send MAX_STREAMS becomes unconditionally
true. This change fixes that so that MAX_STREAMS is only sent when 50% or more streams
have completed.
@ehaydenr ehaydenr requested a review from a team as a code owner November 13, 2025 18:26
Comment thread quiche/src/stream/mod.rs
Comment thread quiche/src/stream/mod.rs Outdated
Comment thread quiche/src/stream/mod.rs Outdated
@ghedo ghedo enabled auto-merge (squash) November 20, 2025 09:59
Copy link
Copy Markdown
Contributor

@antoniovicente antoniovicente left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this amazing fix. And sorry for not noticing the full impact of this change until recently.

@ghedo ghedo merged commit 833e040 into cloudflare:master Feb 16, 2026
26 checks passed
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.

3 participants