On Thu, 28 May 2026 13:24:47 GMT, Matthew Donovan <[email protected]> wrote:
>> This timeout may be caused by different code paths used to resolve
>> `InetAddress.getLoopbackAddress()` in the server thread and
>> `factory.createSocket("localhost",...)` in the client thread. I updated the
>> test to use getLoopbackAddress() for both threads.
>>
>>
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> 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 seven additional
> commits since the last revision:
>
> - changed to use Utils.adjustTimeout()
> - Merge branch 'master' into disable-cipher-suites
> - reverted a bunch of changes; changed to use timeout factor for accept()
> call
> - Merge branch 'master' into disable-cipher-suites
> - 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
I had this patch in our test queue for 4 days now and have not seen any issue
with it.
-------------
Marked as reviewed by azeller (Author).
PR Review: https://git.openjdk.org/jdk/pull/30680#pullrequestreview-4408269159
