On Mon, 26 Jun 2023 17:38:04 GMT, Matthew Donovan <[email protected]> wrote:
> In this PR, I updated the test to use read time-outs. The test is restarted
> if the read operations time-out within (default) 30 seconds. The test makes 5
> attempts before giving up.
Aside from the nit, looks good to me.
test/jdk/javax/net/ssl/DTLS/DTLSWontNegotiateV10.java line 51:
> 49: private static final String DTLSV_1_2 = "DTLSv1.2";
> 50:
> 51: private static final int READ_TIMEOUT_SECS =
> Integer.getInteger("readtimeout", 30);
Nit on line 37 (since I can't flag that directly): Maybe add 8310070 to the
@bug line.
-------------
Marked as reviewed by jnimeh (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/14658#pullrequestreview-1528864782
PR Review Comment: https://git.openjdk.org/jdk/pull/14658#discussion_r1262818408
