On Fri, 20 Mar 2026 16:23:45 GMT, Hai-May Chao <[email protected]> wrote:
> The SSLSocketSSLEngineCloseInbound.java test runs with different protocols, > namely TLSv1.3, TLSv1.2, TLSv1.1, TLSv1, and TLS. The test log file shows > that it completed successfully for the first four protocols, but it failed > when running with protocol=TLS. Protocol TLS is resolved to TLSv1.3. The > failure in the test is a timing-related issue, that server continues with > write() after the client has already closed its side of the connection. This > change removes the race condition by synchronizing the client and server with > CountDownLatch. > > Have validated the change to ensure the test to pass (with > JTREG=REPEAT_COUNT=50). I like the use of CountDownLatch instead of sleep! Thumbs up. ------------- Marked as reviewed by dcubed (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/30340#pullrequestreview-3982793146
