On Tue, 1 Apr 2025 19:11:02 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:
>> Adds `HttpResponse::connectionLabel` method that provides an identifier for >> the connection. >> >> **Implementation note:** The feature is facilitated by >> `HttpConnection::label`, which should not be confused with >> `HttpConnection::id`. This distinction is explained in the JavaDoc of both >> properties. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Apply review suggestions src/java.net.http/share/classes/jdk/internal/net/http/HttpConnection.java line 100: > 98: * {@link PlainHttpConnection} it wraps will share the same label. As > a > 99: * result, compared to {@link #id}, this label does not give a total > order > 100: * among instances. Suggestion: /** * A label to identify the connection. * <p> * This label helps with associating multiple components participating in a * connection. For instance, an {@link AsyncSSLConnection} and the * {@link PlainHttpConnection} it wraps will share the same label. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2024608425