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