RFR: 8373538: Migrate all tests to null-safe "SimpleSSLContext" methods [v3]
Volkan Yazici
vyazici at openjdk.org
Wed Jan 7 09:25:04 UTC 2026
On Wed, 7 Jan 2026 09:07:26 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:
>> 1. [JDK-8372661] introduced null-safe static factory methods to `SimpleSSLContext`
>> 2. [JDK-8373515] migrated `test/jdk/java/net/httpclient/` to these new methods
>> 3. [JDK-8373537] migrated `test/jdk/com/sun/net/httpserver/` to these new methods
>> 4. This PR migrates all remaining `SimpleSSLContext` usages, *and* removes nullable `SimpleSSLContext` factory methods
>>
>> [JDK-8372661]: https://bugs.openjdk.org/browse/JDK-8372661
>> [JDK-8373515]: https://bugs.openjdk.org/browse/JDK-8373515
>> [JDK-8373537]: https://bugs.openjdk.org/browse/JDK-8373537
>
> Volkan Yazici has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains nine commits:
>
> - Update copyright years
> - Migrate forgotten `ClearTextServerSSL`
> - Merge remote-tracking branch 'upstream/master' into simpleSslRest
> - Remove wildcard imports
> - Merge remote-tracking branch 'upstream/master' into simpleSslRest
> - Restore all non-`httpclient`, non-`httpserver` changes in `test/`
> - Make sure `get()` returns the same instance
> - Reverted all changes and only kept `SimpleSSLContext` enhancements
> - Overhaul `SimpleSSLContext` and its usages
test/jdk/com/sun/net/httpserver/ClearTextServerSSL.java line 1:
> 1: /*
This should have been addressed in [JDK-8373537], but forgotten. Doing it here.
[JDK-8373537]: https://bugs.openjdk.org/browse/JDK-8373537
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28905#discussion_r2667658921
More information about the security-dev
mailing list