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

[Daniel Fuchs]

On Mon, 24 Nov 2025 10:21:38 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/Http2TerminationCause.java line 268:
>> 
>>> 266:     private static final class InternalError extends Http2TerminationCause {
>>> 267:         private InternalError(final Throwable cause) {
>>> 268:             super(ErrorFrame.INTERNAL_ERROR, new Exception(cause));
>> 
>> why do we need to wrap the throwable here?
>
> I don't remember why I did that this way, but it looks like a leftover from my experiments. I've updated the PR to remove this wrapping. Thank you for catching this.

Good catch. Wrapping in plain Exception smells like debugging experiment :-)

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

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