> 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
Michael McMahon has updated the pull request incrementally with one additional commit since the last revision: small test update ------------- Changes: - all: https://git.openjdk.org/jdk/pull/9755/files - new: https://git.openjdk.org/jdk/pull/9755/files/5d028bdb..3579c8ce Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=9755&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9755&range=01-02 Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 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
