On Tue, 21 Apr 2026 19:08:16 GMT, Volkan Yazici <[email protected]> wrote:
> Ensure `HttpClient::available` returns `false` if the connection is not > established or the associated socket is found to contain stale data to be > read. Note that this verification is only performed for "streaming POST" > requests. > > --------- > - [X] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). test/jdk/sun/net/www/http/HttpClient/IsAvailable.java line 1: > (failed to retrieve contents of file, check the PR for context) This test was very minimal and is replaced with `StreamingPostHttpClientReuseTest`. test/jdk/sun/net/www/http/HttpClient/StreamingPostHttpClientReuseTest.java line 59: > 57: /* > 58: * @test > 59: * @bug 8009650 8180483 8009650 is copied from the deleted `IsAvailable`. test/jdk/sun/net/www/http/HttpClient/StreamingPostHttpClientReuseTest.java line 333: > 331: // socket `read()` using 1ms timeout. Hence, we give the > 332: // transfer some time for the data to get delivered. > 333: Thread.sleep(adjustTimeout(500)); I'd appreciate it if reviewers can pay attention to this part. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30859#discussion_r3119765526 PR Review Comment: https://git.openjdk.org/jdk/pull/30859#discussion_r3119768304 PR Review Comment: https://git.openjdk.org/jdk/pull/30859#discussion_r3119772565
