RFR: 8378379: Remove reference to obsolete jdk.net.usePlainSocketImpl property from SSLSocketReset test
Jaikiran Pai
jpai at openjdk.org
Thu Feb 26 14:33:28 UTC 2026
On Mon, 23 Feb 2026 01:30:24 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this test-only change which removes the reference to the obsolete `jdk.net.usePlainSocketImpl` property?
>
> `jdk.net.usePlainSocketImpl` was removed in https://bugs.openjdk.org/browse/JDK-8253119 and plays no role anymore in the JDK. This `test/jdk/sun/security/ssl/SSLSocketImpl/SSLSocketReset.java` test was introduced to reproduce a bug in the implementation of PlainSocketImpl, but doesn't rely on any internal socket implementation details, so I think it should be OK to retain this test and just remove the usage of the property. The property has been playing no role in this test, and the test has been functional, ever since the property was removed in JDK-8253119.
Sean rightly noted that this test uses `SSLContext.getDefault()` so retaining the `othervm` would prevent unexpected issues. I've updated the PR accordingly.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/29872#issuecomment-3966985538
More information about the security-dev
mailing list