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