RFR: 8372147: ConnectionFlowControlTest should use HttpResponse.connectionLabel()

Volkan Yazici vyazici at openjdk.org
Thu Nov 27 13:10:09 UTC 2025


On Thu, 27 Nov 2025 13:03:28 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:

> Use `HttpResponse::connectionLabel` instead of `Http2TestExchange::connectionKey` to identify the connection request-response exchange is performed on, for the latter can accidentally return the same identifier again, e.g., if a closed connection is immediately reused.

test/jdk/java/net/httpclient/http2/ConnectionFlowControlTest.java line 1:

> 1: /*

`@bug` is intentionally not extended with [JDK-8372147], since the test does not verify a problem reported in that ticket.

[JDK-8372147]: https://bugs.openjdk.org/browse/JDK-8372147

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28534#discussion_r2568601469


More information about the net-dev mailing list