On Fri, 22 Nov 2024 17:29:23 GMT, Bradford Wetmore <wetm...@openjdk.org> wrote:
>> Artur Barashev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Adding CountDownLatch mechanism
>
> test/jdk/sun/security/ssl/SSLCipher/SSLSocketNoServerHelloClientShutdown.java
> line 74:
>
>> 72: float timeoutFactor = Float.parseFloat(
>> 73: System.getProperty("test.timeout.factor", "1.0"));
>> 74: serverReadTimeout = (int) (30000 * timeoutFactor);
>
> My suggestion was to remove the SoTimeout on both the client and server, also
> including the one now at line 95.
This is just a fallback mechanism right now, otherwise the test can hang for a
very long time if something goes wrong. If 30 seconds is not enough for the
test to run then we fail.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22263#discussion_r1854470354
