On Mon, 6 Jan 2025 05:43:31 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

> Can I please get a review of this change which proposes to address the 
> test-only issue noted in https://bugs.openjdk.org/browse/JDK-8347000?
> 
> As noted in that issue, the test issues HTTP requests with `Content-Length` 
> set to `0` implying no request body. However, the code unintentionally sends 
> additional bytes (14 bytes each request) as part of the request. The JDK's 
> `HttpServer` implementation currently doesn't cause errors for such requests 
> (although the HTTP/1.1 RFC expects an error to be raised). 
> 
> The change in this PR merely addresses the test code to not send these 
> additional bytes. The test continues to pass after this change.

This pull request has now been integrated.

Changeset: 5e6cda47
Author:    Jaikiran Pai <j...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/5e6cda4799a6bf12370bc6a04b218ebed32dee53
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8347000: Bug in com/sun/net/httpserver/bugs/B6361557.java test

Reviewed-by: msheppar, michaelm

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

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

Reply via email to