RFR: 8371903: HttpClient: improve handling of HTTP/2 GOAWAY frames with error code [v7]

Jaikiran Pai jpai at openjdk.org
Tue Dec 16 10:30:35 UTC 2025


On Tue, 16 Dec 2025 10:25:04 GMT, EunHyunsu <duke at openjdk.org> wrote:

>> Thank you @ehs208 for these updates. The changes look good to me. I'll run the latest changes in our CI to make sure this continues to pass.
>> 
>> On a related note, have you run tier2 locally with these changes? The GitHub actions job only run tier1 and networking tests aren't covered in that. It would be good to run tier2 on your local setup too - that way any future contributions in this area can be tested locally to verify that changes don't break unexpected existing tests.
>
> @jaikiran I ran jdk_net tests locally. Results: 1047 passed, 1 failed.
> 
>   The failed test is `HTTPSetAuthenticatorTest` (HTTP/1.1 Authenticator test) with `SocketException: Connection reset`, which appears to be a flaky test unrelated to this PR's HTTP/2 GOAWAY changes. All HTTP/2 related tests passed successfully.

Hello @ehs208, I'm investigating a intermittent test failure of the newly added test in our CI. I think the test will need a few adjustments. I haven't had a chance to go over the failure in its entirety. I will add some details here when I am done looking at that failure.

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

PR Comment: https://git.openjdk.org/jdk/pull/28632#issuecomment-3659837419


More information about the net-dev mailing list