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

[Michael McMahon]

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