On Tue, 21 Jul 2026 07:59:41 GMT, Daniel Jeliński <[email protected]> wrote:

> Fix the test so that it exercises the max request time again.
> 
> The test is supposed to check if the server closes the connection when it 
> receives some bytes from the client, but the bytes do not form a complete 
> request before the timeout elapses. Until recently, the server-side request 
> parser was looking for a CRLF sequence before checking if the received bytes 
> form a request, and the TLS handshake was a good enough approximation of an 
> incomplete HTTP request. 
> [JDK-8373677](https://bugs.openjdk.org/browse/JDK-8373677) modified the 
> parser to reject TLS early.
> 
> This PR modifies the test to start sending a legal HTTP request, but stop 
> sending before the final CRLF sequence is sent,
> 
> The test continues to pass.
> 
> ---------
> - [X] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

This pull request has now been integrated.

Changeset: 13cb51b0
Author:    Daniel Jeliński <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/13cb51b07d3cd454a1c917c8674ee8991e92ce72
Stats:     44 lines in 1 file changed: 17 ins; 4 del; 23 mod

8388538: com/sun/net/httpserver/simpleserver/jwebserver/MaxRequestTimeTest.java 
does not test the maximum request time

Reviewed-by: jpai

-------------

PR: https://git.openjdk.org/jdk/pull/31983

Reply via email to