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

[EunHyunsu]

On Wed, 10 Dec 2025 13:43:18 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> @djelinski Fixed the intermittent test failures:
>>   - Removed `close()` call after GOAWAY to avoid connection reset errors
>>   - Server now relies on Http2TestServer for connection cleanup
>>   - Tested 10 times locally, all passed
>> 
>>   Should be stable now. Please review when you get a chance
>
> @ehs208  thanks for that, the test looks stable now.
> 
> Looking at the test logs, I realized why the `goAwaySentLatch.await` was where you originally had it; now that it's after all 3 requests, the requests are sent on 3 distinct connections, which is not what we want. Please move it back to its original position (after sending first request). Sorry for my confusion.

@djelinski Oops, you're right! I moved `goAwaySentLatch.await()` back to its original position and now it works as expected - only 1 connection is created initially, and the other 2 are retries. Tested locally and all passed.

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

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