Netty: respond with 408 when the request timeout fires before the body is fully received - #5466

Merged

adamw merged 41 commits into

fix/return-400-if-not-all-body-bytes-received

Sep 24, 2026

Commits on Sep 23, 2026

Add test reproducing the issue of invalid response
rwalerow
committed
8b72e79
Change test to plain java socket implementation
rwalerow
committed
d120bb3
Add request body completed tracker to track whether entire body was already read
rwalerow
committed
b45cbe6
Rename Body tracker to express intention
rwalerow
committed
2ad1f49
Improve the way user events are handled
rwalerow
committed
f0dd8e9
Fix multiple event branches call on Idle event and add missing connection close header
rwalerow
committed
3d36b48
Randomize port in a new test and lower the timeouts
rwalerow
committed
1b151a8
Report 400 only on READ timeouts with uncomplete body send
rwalerow
committed
c9082df
Add a new behavior description to the netty conifg
rwalerow
committed
6586092
Reformat code
rwalerow
committed
7bce660
Remove unused import and rename method to be more intuitive
rwalerow
committed
3908f64
Unify error write methods and change logging type for not fully send request
rwalerow
committed
429b054
Rephrase param documentation
rwalerow
committed
f644f0c
Tracking body completed would work properly also for provided pipelines
rwalerow
committed
eeb50f9
Change test to check whethere 503 was not additionally writen and use Resource to close a socket
rwalerow
committed
9a6c5b5
Request completed tracker makes sens only if request timeout is set
rwalerow
committed
114ed86
Added an information where in the pipeline should RequestBodyCompletedTracker be placed
rwalerow
committed
b7aecaa
Remove unnecesary import
rwalerow
committed
a6b9a6d
Reformat missed files
rwalerow
committed
38168cd
Rephrase requestTimeout documentation
rwalerow
committed
f8d53eb
Rephrase reading debug message, grammar fix
rwalerow
committed
35850c8
Open up Request completed Tracker
rwalerow
committed
67f072e
Remove unnecesary wait witihn a test case
rwalerow
committed
060b819
Add so timeout to socket
rwalerow
committed
da05efe
Find and remove request body tracker when upgrading connection to a websocket
rwalerow
committed
75c63b5
Reorganize imports
rwalerow
committed
a7ff811
Fix renaming import
rwalerow
committed
ed782d2
Use stateful handler instead of channle attribute
rwalerow
committed
96529ba
Return 408 instead of 400 in incomplete request
rwalerow
committed
345cf0c
Move docs to 408 status and remove unused import
rwalerow
committed
c23839f
Rename body completed tracker and rollback to single timeout to avoid double notification
rwalerow
committed
69232bc
Remove unused variables and imports
rwalerow
committed
c6332a9
Additional comments and removed unnecessary sleep in test
rwalerow
committed
1db0ff0
Remove unused variable
rwalerow
committed
67dc5dc
Fixes after a next round
rwalerow
committed
5345881
Review fixes and test refactor

rwalerow
and
claude
committed
90b8e9a
Extend comment
rwalerow
committed
a5952ef
Extend documentation
rwalerow
committed
4e22608
Address latest CR issues
rwalerow
committed
fbb3b64
Rearrange variables
rwalerow
committed
39f1495

Commits on Sep 24, 2026

Netty: install the request body tracker from NettyServerHandler, keep it internal
adamw
committed
5d2f893