Uh oh!
There was an error while loading. Please reload this page.
[fix](web) FE startup fail due to websocket startup - #60369
Conversation
Thearas
commented
Jan 29, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
gavinchou
commented
Jan 29, 2026
run buildall |
doris-robot
commented
Jan 29, 2026
TPC-H: Total hot run time: 32085 ms |
doris-robot
commented
Jan 29, 2026
ClickBench: Total hot run time: 28.12 s |
hello-stephen
commented
Jan 29, 2026
FE Regression Coverage ReportIncrement line coverage |
nsivarajan
commented
Feb 6, 2026
@CalvinKirs — could you please assist with reviewing and merging this? |
nsivarajan
commented
Feb 10, 2026
@gavinchou@CalvinKirs - could you please assist with reviewing and merging this? |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
nsivarajan
commented
Feb 16, 2026
@gavinchou could you please assist with reviewing and merging this? |
nsivarajan
commented
Feb 25, 2026
@gavinchou@morningman could you please assist with reviewing and merging this? |
Uh oh!
There was an error while loading. Please reload this page.
## Summary Pick #60369 to branch-4.1 ## Changes - Fix FE startup failure when enable_https=true by using Spring Boot's addServerCustomizers() hook to initialize WebSocket support before WebAppContext starts - Compatible with branch-4.1's existing HTTP header size configuration ## Test - [x] FE build passed ## Related Issue close#60366 Co-authored-by: Sivarajan Narayanan <narayanan_sivarajan@apple.com>
What problem does this PR solve?
Issue Number: close#60366
Related PR: #57710
Problem Summary:
Apache Doris Frontend fails to start when enable_https=true with the following error:
RCA for failure : Jetty 12 introduced strict lifecycle state validation for WebSocket initialization.
Fix:
Use Spring Boot's addServerCustomizers() hook which runs before the WebAppContext starts, combined with server.getDescendant(WebAppContext.class) to safely navigate Jetty 12's handler tree.Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)