Skip to content

branch-4.0: [fix](http) Fix jetty_server_max_http_header_size not applied in Jetty 12 (#61197) - #61903

Merged
yiguolei merged 1 commit into
apache:branch-4.0from
liaoxin01:pick-pr-61197-to-branch-4.0
Apr 15, 2026
Merged

branch-4.0: [fix](http) Fix jetty_server_max_http_header_size not applied in Jetty 12 (#61197)#61903
yiguolei merged 1 commit into
apache:branch-4.0from
liaoxin01:pick-pr-61197-to-branch-4.0

Conversation

@liaoxin01

@liaoxin01liaoxin01 commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Pick #61197

@liaoxin01wesleyliao
[fix](http) Fix jetty_server_max_http_header_size not applied in Jett…
5155e9c
…y 12 (apache#61197)
HTTP 431 "Request Header Fields Too Large" error occurs when sending
requests with large headers (>8KB) to FE, even though
`jetty_server_max_http_header_size` is configured to 10MB.
This issue appears in cloud-4.1.2 (Jetty 12 + Spring Boot 3) but not in
cloud-4.0.10 (Jetty 9 + Spring Boot 2).
In Jetty 12 with Spring Boot 3, the `server.max-http-header-size`
property does not automatically apply to custom `HttpConfiguration`
objects created in `WebServerFactoryCustomizer`. The default 8KB limit
remains in effect.
Explicitly set `requestHeaderSize` on all server connectors in
`WebServerFactoryCustomizerConfig` by adding a server customizer that
configures the `HttpConfiguration` for each connector.
- Added regression test `test_large_http_header.groovy` that verifies
100KB headers are accepted
- Verified before fix: HTTP 431 error with large headers
- Verified after fix: HTTP 200 success with large headers
CopilotAI review requested due to automatic review settings March 30, 2026 14:22
@Thearas

Thearas commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

CopilotAI reviewed Mar 30, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR backports a fix to ensure jetty_server_max_http_header_size is actually applied when running FE HTTP service on Jetty 12, and adds a regression test to validate large request headers are accepted.

Changes:

  • Customize Jetty connectors to apply Config.jetty_server_max_http_header_size via HttpConfiguration#setRequestHeaderSize.
  • Add a regression test that sends a ~100KB HTTP request header to /api/health and verifies it doesn’t fail with 431.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

FileDescription
fe/fe-core/src/main/java/org/apache/doris/httpv2/config/WebServerFactoryCustomizerConfig.javaApplies request header size to existing Jetty connectors via a server customizer.
regression-test/suites/http_p0/test_large_http_header.groovyNew regression test exercising large HTTP request headers against FE’s /api/health.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadregression-test/suites/http_p0/test_large_http_header.groovy
Comment threadfe/fe-core/src/main/java/org/apache/doris/httpv2/config/WebServerFactoryCustomizerConfig.java
Comment threadregression-test/suites/http_p0/test_large_http_header.groovy
@liaoxin01

liaoxin01 commented Mar 30, 2026

Copy link
Copy Markdown
ContributorAuthor

run buildall

@hello-stephen

hello-stephen commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 80.00% (8/10) 🎉
Increment coverage report
Complete coverage report

@yiguolei
yiguolei merged commit df937b2 into apache:branch-4.0Apr 15, 2026
30 of 33 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@yiguoleiyiguoleiyiguolei approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@liaoxin01@Thearas@hello-stephen@yiguolei