RFR: 8292177: InitialSecurityProperty JFR event [v3]

Sean Coffey coffeys at openjdk.org
Mon Oct 10 19:23:52 UTC 2022


On Mon, 10 Oct 2022 14:19:18 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> src/java.base/share/classes/java/security/ProtectionDomain.java line 76:
>> 
>>> 74:     static class JavaSecurityAccessImpl implements JavaSecurityAccess {
>>> 75:         /* cache a copy for recording purposes */
>>> 76:         static Properties initialSecurityProperties;
>> 
>> This doesn't look very clean. Could the Security class hold the initial security properties and provide an accessor method that JavaSecurityAccess:getIinitialProperties could use?
>
> Agree, and alternatively, it seems cleaner to add a new SharedSecrets class for `java.security.Security` and remove the dependency on PD.

modified code to have Security class hold the initial properties and provided an accessor method

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

PR: https://git.openjdk.org/jdk/pull/10394



More information about the security-dev mailing list