On Wed, 9 Jul 2025 00:03:41 GMT, Anthony Scarpino <ascarp...@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

Reply via email to