Skip to content

[26.7] Align the HTTP/2 max header list size with the HTTP/1.1 max header si… - #51397

Merged
vmuzikar merged 1 commit into
keycloak:release/26.7from
shawkins:backport-51246-26.7
Aug 6, 2026
Merged

[26.7] Align the HTTP/2 max header list size with the HTTP/1.1 max header si…#51397
vmuzikar merged 1 commit into
keycloak:release/26.7from
shawkins:backport-51246-26.7

Conversation

@shawkins

@shawkins shawkins commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

…ze (#51246)

Keycloak configures quarkus.http.limits.max-header-size=65535 for HTTP/1.1 but never sets max-header-list-size so HTTP/2 requests were held to the much smaller Quarkus default, as noted in the original issue. Any request with headers between the two limits succeeded over HTTP/1.1 but was rejected at the HTTP/2 codec with no HTTP response and nothing logged. This aligns the HTTP/2 limit with HTTP/1.1 and adds an integration test covering large headers over both protocols.

Closes #51182

(cherry picked from commit a3a37df)

@vmuzikar
vmuzikar enabled auto-merge (squash) August 3, 2026 14:55
(keycloak#51246)

Keycloak configures quarkus.http.limits.max-header-size=65535 for
HTTP/1.1 but never sets max-header-list-size so HTTP/2 requests were
held to the much smaller Quarkus default, as noted in the original
issue. Any request with headers between the two limits succeeded over
HTTP/1.1 but was rejected at the HTTP/2 codec with no HTTP response and
nothing logged. This aligns the HTTP/2 limit with HTTP/1.1 and adds an
integration test covering large headers over both protocols.

Closes keycloak#51182

Signed-off-by: James Gore <83005220+thejamesgore@users.noreply.github.com>

(cherry picked from commit a3a37df)
@shawkins
shawkins force-pushed the backport-51246-26.7 branch from 8f5eba5 to 22e6826 Compare August 4, 2026 13:17
@vmuzikar
vmuzikar merged commit 506df74 into keycloak:release/26.7 Aug 6, 2026
87 checks passed
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.

4 participants