Integrated: 8249825: Tests sun/security/ssl/SSLSocketImpl/SetClientMode.java and NonAutoClose.java marked with @ignore

Fernando Guallini fguallini at openjdk.org
Wed May 14 08:42:06 UTC 2025


On Tue, 4 Mar 2025 13:07:37 GMT, Fernando Guallini <fguallini at openjdk.org> wrote:

> The following tests are marked with @ignore (not running):
> 
> - sun/security/ssl/SSLSocketImpl/SetClientMode.java: it checks that setting the clientMode after the handshake has begun is not permitted, but this was failing intermittently due to a race condition, it was possible that SetClientMode was called before the client socket was updated with handshake isNegotiated = true. The fix is to introduce a latch to sync between client and main threads. Included additional refactoring to ensure test stability.
> 
> - sun/security/ssl/SSLSocketImpl/NonAutoClose.java: This test should only run in TLS <= 1.2, as TLSv1.3 changes the behaviour of close_notify. Included additional refactoring to ensure test stability.
> 
> Executed both tests 10K times, no test flakiness found

This pull request has now been integrated.

Changeset: 86f39ab5
Author:    Fernando Guallini <fguallini at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/86f39ab5bb58a1d1e720badd7261365398134151
Stats:     391 lines in 2 files changed: 26 ins; 231 del; 134 mod

8249825: Tests sun/security/ssl/SSLSocketImpl/SetClientMode.java and NonAutoClose.java marked with @ignore

Reviewed-by: abarashev, rhalade

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

PR: https://git.openjdk.org/jdk/pull/23898


More information about the security-dev mailing list