On Mon, 26 Jun 2023 17:38:04 GMT, Matthew Donovan <mdono...@openjdk.org> 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