RFR: 8350279: HttpClient: Add a new HttpResponse method to identify connections [v6]
Daniel Fuchs
dfuchs at openjdk.org
Fri Apr 4 16:43:50 UTC 2025
On Fri, 4 Apr 2025 16:36:38 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Good point - our default idle timeout is 30s IIRC. There would need to be a pause of 30s between the two calls to `client.send` however. Not impossible but unlikely. We could either pass a higher timeout (with e.g. `-Djdk.httpclient.keepalive.timeout=120`) preemptively, or wait until we see the test fail...
>
> Waiting for the test to fail due to this reason and then adjusting the timeout is fine with me.
On the plus side - if we add the `-Djdk.httpclient.keepalive.timeout=120` now we might not have to deal with it later, and maybe it will give us a clue if the test ever fails due to this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24154#discussion_r2029133030
More information about the net-dev
mailing list