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