Skip to content

fix(http2): bound header size to 256 KiB - #1978

Open
brianquinlan wants to merge 5 commits into
dart-lang:masterfrom
brianquinlan:unbound_continuation
Open

fix(http2): bound header size to 256 KiB#1978
brianquinlan wants to merge 5 commits into
dart-lang:masterfrom
brianquinlan:unbound_continuation

Conversation

@brianquinlan

@brianquinlan brianquinlan commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Add maxHeaderListSize setting to ClientSettings and ServerSettings (defaulting to 256 KiB), advertise SETTINGS_MAX_HEADER_LIST_SIZE, enforce it during HPACK decoding, and limit CONTINUATION frame buffering with $O(N)$ defragmentation.


  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Note: The Dart team is trialing Gemini Code Assist. Don't take its comments as final Dart team feedback. Use the suggestions if they're helpful; otherwise, wait for a human reviewer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant