On Thu, 4 Aug 2022 20:41:33 GMT, Michael McMahon <micha...@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 <micha...@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
