rschmitt commented on PR #547: URL: https://github.com/apache/httpcomponents-core/pull/547#issuecomment-3176730639
Do you have the corresponding client changes? I tried integrating this into `PoolingAsyncClientConnectionManager` after the fashion of the H2 `PingCommand` code path, and now my requests don't finish. I think there's might be a race condition here involving command execution and connection closure. I remember a few years ago there was an issue where enabling inactive connection validation would cause the client to hang by submitting a `PingCommand` on a connection that had already received a `GOAWAY`. My records indicate that this bug was fixed, but now I'm not so sure, or maybe it was reintroduced. In the debugger, I don't even see the IOReactor wake up when the command is submitted; `readyCount` comes back as `0`. It doesn't look like there's anything in `IOSessionImpl::enqueue` that prevents a `Command` from being submitted against an already-closed connection. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org