On Wed, 10 Dec 2025 14:35:29 GMT, Daniel Jeliński <[email protected]> wrote:
>> EunHyunsu has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8371903: Move goAwaySentLatch.await() back to original position >> >> Move the await call to after the first request but before the second >> and third requests, ensuring only one initial connection is created >> and the other requests are properly retried. > > Right, we only create one connection initially, and requests 2 and 3 are > usually sent over the first connection before they are retried on new > connections. This can be observed in the server logs as `resetting stream 3 > as REFUSED_STREAM`. Ideally the server should not send anything after sending > the GOAWAY frame with error, but I think what we have is good enough, @djelinski Thanks for the explanation. That makes sense. If there’s an opportunity in the future, I’d be happy to help clean that part up as well. Appreciate your guidance. ------------- PR Comment: https://git.openjdk.org/jdk/pull/28632#issuecomment-3637412226
