On Fri, 5 Aug 2022 10:32:56 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 > > Michael McMahon has updated the pull request incrementally with one > additional commit since the last revision: > > update from Jai's review Thank you for the changes, Michael. The updated version looks fine to me. ------------- Marked as reviewed by jpai (Reviewer). PR: https://git.openjdk.org/jdk/pull/9755
