RFR: 8278398: jwebserver: Add test to confirm maximum request time [v2]

Michael McMahon michaelm at openjdk.java.net
Fri Jan 14 12:41:26 UTC 2022


On Thu, 13 Jan 2022 15:39:28 GMT, Julia Boes <jboes at openjdk.org> wrote:

>> This new test confirms that the jwebserver does not wait indefinitely for a request to arrive, but instead closes the connection when the maximum request time is reached. To facilitate this, Exchange::run is amended to process cancelled keys in the case where the current thread is the dispatcher thread (which is the case for the jwebserver).
>> 
>> Testing: tier 1-3 all clear.
>
> Julia Boes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   address PR comments:
>   * remove redundant null check
>   * use try-finally to stop process in test

LGTM

-------------

Marked as reviewed by michaelm (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7052


More information about the net-dev mailing list