On Tue, 22 Nov 2022 11:10:14 GMT, Conor Cleary <[email protected]> wrote:
> Proposed changes to reduce the default Keep Alive Timeout value in > `jdk/internal/net/http/HttpClientImpl.java` from 1200 seconds to 30 seconds. > The current default value of 1200s is needlessly high and in remote > connections that allow a client to suggest a value, this will mean that idle > connections will wait very long before being dropped. > > This value of 30 seconds was chosen by reviewing the current default values > for Keep-Alive timeouts in common server implementations and choosing a value > that would have the client's default value be lower whenever the value can be > set. > > Further discussion on this change can be seen the pull request [8288717: Add > a means to close idle connections in HTTP/2 connection pool > #10183](https://github.com/openjdk/jdk/pull/10183). This was never implemented. Note that KeepAlive is only valid for HTTP/1.1 - not HTTP/2 See https://datatracker.ietf.org/doc/html/rfc9113#section-8.2.2. I have logged an RFE https://bugs.openjdk.org/browse/JDK-8326160 ------------- PR Comment: https://git.openjdk.org/jdk/pull/11285#issuecomment-1952159400
