RFR: 8345060: Remove Security Manager dependencies from java.security.KeyStore and Identity APIs and implementations [v2]

Alan Bateman alanb at openjdk.org
Thu Nov 28 16:15:40 UTC 2024


On Wed, 27 Nov 2024 14:39:21 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Now that JEP 486 has been integrated, `java.security.KeyStore` and `Identity` API and impl dependencies on `System.getSecurityManager`, `AccessController.doPrivileged` and `AccessControlContext` can be removed.
>> 
>> Most of the changes are straightforward, although some of the refactoring in `KeyStore` was a bit complex due to having to extract the code from the `PrivilegedAction` subclasses.
>
> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix infinite loop which was causing a few test failures.

I went through the changes, focused on KeyStore mostly, and looks okay.

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

Marked as reviewed by alanb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22401#pullrequestreview-2468531581


More information about the security-dev mailing list