RFR: 8358942: HttpClient adds Content-Length: 0 for a GET request with a BodyPublishers.noBody() [v4]

Daniel Fuchs dfuchs at openjdk.org
Tue Oct 14 18:15:30 UTC 2025


On Tue, 14 Oct 2025 18:00:20 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> Do not send the Content-Length header on HTTP/1.1 requests when the content length is known to be zero and the method does not expect content or is unknown.
>> 
>> This brings the HTTP/1.1 implementation in line with the recommendations from RFC 9110.
>> 
>> The existing ContentLengthHeaderTest was extended to cover the modified scenarios.
>> 
>> Tier1-3 tests continue to pass.
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
> 
>   More strict verification of content-length handling

test/jdk/java/net/httpclient/ContentLengthHeaderTest.java line 224:

> 222:         assertEquals(resp.statusCode(), 200, resp.body());
> 223:         assertEquals(resp.version(), version);
> 224:         assertEquals(resp.body(), "Request completed");

Ok - so now: remove or add everywhere?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27727#discussion_r2430042438


More information about the net-dev mailing list