On Tue, 7 Dec 2021 10:24:31 GMT, Julia Boes <jb...@openjdk.org> wrote:

> This change sets a maximum request time for the `jwebserver` (and `java -m 
> jdk.httpserver`), unless it has already been set. 
> 
> While here, I removed a comment from the implementation classes, a left-over 
> from the pre-module era that is not needed and should not have been included 
> in the first place.
> 
> Testing: tier 1-3, and manual testing to confirm a request fails after the 
> given time, e.g.:
> 
> $ curl https://127.0.0.1:8000
> curl: (35) LibreSSL SSL_connect: SSL_ERROR_SYSCALL in connection to 
> 127.0.0.1:8000

LGTM. 
Might be good to have at least a manual test to verify that for ATR - I'm OK if 
this is done in a followup PR

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

Marked as reviewed by dfuchs (Reviewer).

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

Reply via email to