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 ------------- Commit messages: - fix plus test Changes: https://git.openjdk.org/jdk/pull/9755/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9755&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8291637 Stats: 135 lines in 2 files changed: 135 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/9755.diff Fetch: git fetch https://git.openjdk.org/jdk pull/9755/head:pull/9755 PR: https://git.openjdk.org/jdk/pull/9755
