Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cap H2 MAX_HEADER_LIST_SIZE according to config #12699

Open
wants to merge 2 commits into
base: jetty-12.0.x
Choose a base branch
from
Open

Cap H2 MAX_HEADER_LIST_SIZE according to config #12699

wants to merge 2 commits into from
+44 −2

Conversation

lorban
Copy link
Contributor

@lorban lorban commented Jan 10, 2025

fixes #12690

@lorban
#12690 cap MAX_HEADER_LIST_SIZE
9a44908 
Signed-off-by: Ludovic Orban <[email protected]>
@lorban lorban requested a review from sbordet January 10, 2025 15:33
@lorban lorban self-assigned this Jan 10, 2025
@lorban lorban added Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement labels Jan 10, 2025
@lorban
#12690 fix test
652583c 
Signed-off-by: Ludovic Orban <[email protected]>
@lorban lorban changed the title cap MAX_HEADER_LIST_SIZE Cap H2 MAX_HEADER_LIST_SIZE according to config Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbordet sbordet sbordet approved these changes

Assignees

@lorban lorban

Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Projects
Jetty 12.0.17
Status: 👀 In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add configurable capping for values of H2 MAX_HEADER_LIST_SIZE settings frames
2 participants
@lorban @sbordet