On 13/08/2024 17:53, Mark Thomas wrote:
On 13/08/2024 09:48, Itzhak Fadida wrote:
Thank you for your reply. I created a repository that demonstrates the issue.

https://github.com/tzahifadida/test-chunked

Thanks. That is very helpful.

git bisect has identified this commit:

https://github.com/apache/tomcat/commit/92e3c7a7adc574a859ab70333bf930561dcf1e9d

as the cause of the change in behaviour.

That makes sense as it is a change to the processing of chunked requests.

I need to debug this further to figure out what and where the root cause is.

Found it. The root cause was in Tomcat. The CRLF terminating a chunk of the request body wasn't accounted for when determining if there was more data to read. This caused the decoder to continue to try and read when there was no data. This in turn caused the decoder to wait until the next line arrived (and the next line, and the next line) before returning causing the whole body at once.

I've written a test case based on your test project and confirmed that the proposed fix addresses the issue.

Mark

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscr...@tomcat.apache.org
For additional commands, e-mail: users-h...@tomcat.apache.org

Reply via email to