Integrated: 8291637: HttpClient default keep alive timeout not followed if server sends invalid value

Michael McMahon michaelm at openjdk.org
Fri Aug 5 14:56:02 UTC 2022


On Thu, 4 Aug 2022 20:41:33 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

> Hi,
> 
> Some new keep alive tests are exposing some old bugs. In this case if the server sends an invalid timeout (say -20 seconds) we accept it creating a timeout in the past. So, the first time the keep alive thread wakes up it will close the connection.
> The correct behavior is to ignore the invalid parameter and fallback to the default timeout or the timeout set by the relevant system property.
> 
> Thanks,
> Michael

This pull request has now been integrated.

Changeset: b17a745d
Author:    Michael McMahon <michaelm at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b17a745d7f55941f02b0bdde83866aa5d32cce07
Stats:     147 lines in 2 files changed: 147 ins; 0 del; 0 mod

8291637: HttpClient default keep alive timeout not followed if server sends invalid value

Reviewed-by: jpai, dfuchs

-------------

PR: https://git.openjdk.org/jdk/pull/9755


More information about the net-dev mailing list