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

[Hai-May Chao]

On Thu, 17 Jul 2025 20:10:54 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:

>> 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?
>
> To clarify, are you asking for a more helpful error message, like "session was not resumed"?  Rather than just the error from `checkResumedSession()`?

Yes. Thanks.

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

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