On Fri, 8 May 2026 00:51:56 GMT, Rajan Halade <[email protected]> wrote:
>> Matthew Donovan has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains three additional >> commits since the last revision: >> >> - updated test to enable server thread to exit quickly if client has >> connection error >> - Merge branch 'master' into disable-cipher-suites >> - 8374454: Test >> sun/security/ssl/CipherSuite/DisabledCipherSuitesNotNegotiated.java from >> JDK-8356544 shows intermittent timeouts > > test/jdk/sun/security/ssl/CipherSuite/DisabledCipherSuitesNotNegotiated.java > line 187: > >> 185: }; >> 186: >> 187: Boolean result = >> executorService.invokeAny(List.of(serverThread, > > invokeAny will return as soon as one task completes that means test would > pass even if server fails after client success. I refactored the test to just use a timeout for the `accept()` call. Hopefully this will be more reliable and not result in unnecessary failures ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30680#discussion_r3242042929
