RFR: 8344366: Remove Security Manager dependencies from javax.net.ssl and sun.security.ssl packages [v2]
Sean Mullan
mullan at openjdk.org
Fri Nov 22 16:43:19 UTC 2024
On Fri, 22 Nov 2024 12:53:14 GMT, Sean Coffey <coffeys at openjdk.org> wrote:
>> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Revert @see text in TrustManagerFactory.
>> Remove sun.security.action.OpenFileInputStreamAction.
>
> src/java.base/share/classes/sun/security/ssl/TrustStoreManager.java line 380:
>
>> 378: if (!"NONE".equals(descriptor.storeName)) {
>> 379: try (FileInputStream fis =
>> 380: new FileInputStream(descriptor.storeFile)) {
>
> wonder if `OpenFileInputStreamAction` could be removed from the codebase ? No other usage detected.. with the exception of test/jdk/sun/security/action/Generify.java which is just a sanity test and could be edited.
Yes, I removed it and updated the test. All the action classes will eventually be removed once all dependencies are gone.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22301#discussion_r1854261278
More information about the net-dev
mailing list