Skip to content

[feature] Increase HTTP header size#141

Merged
Ramon-Jimenez merged 1 commit intodevelopmentfrom
feature/increase-http-header-size
Jun 5, 2025
Merged

[feature] Increase HTTP header size#141
Ramon-Jimenez merged 1 commit intodevelopmentfrom
feature/increase-http-header-size

Conversation

@tokland
Copy link
Copy Markdown
Contributor

@tokland tokland commented Apr 30, 2025 •
edited
Loading

https://app.clickup.com/t/8698whf2e #8698whf2e

The client detected this on cpr-pro, but it will be useful to add it in d2-docker.

Problem: When a request with email/notification has a long message (worsen by the encoding of non-latin languages) , we hit: "<!doctype html><title>HTTP Status 400 – Bad Request".
Solution: Set maxHttpHeaderSize=32k (default: 8k)

java.lang.IllegalArgumentException: Request header is too large
        at org.apache.coyote.http11.Http11InputBuffer.fill(Http11InputBuffer.java:790)
        at org.apache.coyote.http11.Http11InputBuffer.parseHeader(Http11InputBuffer.java:971)
        at org.apache.coyote.http11.Http11InputBuffer.parseHeaders(Http11InputBuffer.java:604)

Note: The typical error we had with URL length is 414, but here incresing the maxSize allows the long URL, so it's a bit weird.

@Ramon-Jimenez Ramon-Jimenez merged commit fd34f2f into development Jun 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tokland @Ramon-Jimenez