ok2c commented on PR #547: URL: https://github.com/apache/httpcomponents-core/pull/547#issuecomment-3187973770
> but I was reminded today that all of the connection persistence headers are [explicitly prohibited](https://httpwg.org/specs/rfc9113.html#ConnectionSpecific) by HTTP/2 and HTTP/3. @rschmitt They are and for a good reason: multiplexing connections cannot permit a single message exchange to shut down the connection potentially shared by many concurrent exchanges. > This means that what is happening in my reproducer is actually pretty representative of what happens on an HTTP/2 connection I cannot agree with that. GOAWAY(NO_ERROR) signals _initiation_ of connection shutdown. A well behaved endpoint should not just write out GOAWAY(NO_ERROR) and immediately drop connection. It can, however, half-close it on its end and stop processing incoming requests which can be safely retried over another connection. Please do add `TestConnectionClosureRace` to the project a test with no asserts. It is certainly useful but I cannot agree it represents a protocol conformant behavior. -- 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
