RFR: 8333857: Test sun/security/ssl/SSLSessionImpl/ResumeChecksServer.java failed: Existing session was used

[Hai-May Chao]

On Wed, 9 Jul 2025 00:03:41 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:

> Hi
> 
> I need a review for a client-server threading change that will hopefully address intermittent failures.  This change is similar to [JDK-8348309](https://bugs.openjdk.org/browse/JDK-8348309).  The functional testing is unchanged.
> 
> thanks
> 
> Tony

test/jdk/sun/security/ssl/SSLSessionImpl/ResumeChecksClient.java line 92:

> 90:             // fail if session is not resumed
> 91:             checkResumedSession(firstSession, secondSession);
> 92:             System.out.println("secondSession used resumption: PASS");

Would you consider to catch the exception from checkRsumedSession() if for some reason resumption fails unexpectedly and log that the test failed due to unexpected session mismatch?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26206#discussion_r2212036431