RFR: 8326498: java.net.http.HttpClient connection leak using http/2 [v7]

[Daniel Fuchs]

On Wed, 19 Nov 2025 13:32:00 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java line 38:
>> 
>>> 36: import java.net.http.HttpHeaders;
>>> 37: import java.nio.ByteBuffer;
>>> 38: import java.nio.channels.NetworkChannel;
>> 
>> Unless I'm mistaken this import is not needed?
>
> The javadoc comment I added to `Http2Connection.isOpen()` uses `{@linkplain NetworkChannel#isOpen() channel is open}`. It looks like the IDE added it for that reason. I'm going to see if removing the import will cause any build failures in any places.

Ah - OK - I missed that - the import is needed then.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28233#discussion_r2542046983