On Tue, 26 Nov 2024 21:06:14 GMT, Sean Mullan <mul...@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.
src/java.base/share/classes/java/security/KeyStore.java line 2091:
> 2089: }
> 2090: } catch (Exception e) {
> 2091: oldException = e;
The updated KeyStore.java code looks good. The run0() is replaced by
getKeyStore0(). Here it does not do e.getCause(), and it looks like because the
exception is not wrapped by PrivilegedExceptionAction anymore?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22401#discussion_r1859637818
