On Mon, 20 Oct 2025 17:40:26 GMT, Volkan Yazici <[email protected]> wrote:
>> Validate and clamp the value populated from the `jdk.httpclient.bufsize` >> system property. Added two new tests: Does clamping work? Does the minimum >> value wedge the client? > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Apply property doc suggestion > > Co-authored-by: Daniel Fuchs <[email protected]> test/jdk/java/net/httpclient/BufferSize1Test.java line 108: > 106: if (response.statusCode() != 200) { > 107: throw new AssertionError("Was expecting status code > 200, found: " + response.statusCode()); > 108: } It would be good to send at least two requests. I am not sure the test tests what it says it does - the HEAD request has no bodies in either ways. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27874#discussion_r2445707950
