RFR: 8344248: Remove Security Manager dependencies from java.security.jgss and jdk.security.jgss modules
Andrey Turbanov
aturbanov at openjdk.org
Thu Nov 21 06:52:19 UTC 2024
On Tue, 19 Nov 2024 20:43:25 GMT, Sean Mullan <mullan at openjdk.org> wrote:
> Now that JEP 486 has been integrated, `java.security.jgss` and `jdk.security.jgss` module dependencies on `System.getSecurityManager` and `AccessController.doPrivileged*` can be removed.
>
> There is an undocumented property named "sun.security.krb5.autodeducerealm" that can probably be removed as it only affected behavior when an SM was enabled; however I have left the code that reads the property as-is (see sun/security/krb5/Realm.java) and have opened a separate issue to determine if it can be safely removed.
src/java.security.jgss/share/classes/sun/security/jgss/wrapper/SunNativeProvider.java line 115:
> 113: }
> 114: Oid[] mechs = GSSLibStub.indicateMechs();
> 115: HashMap<String,String> map = new HashMap<>();
Suggestion:
HashMap<String, String> map = new HashMap<>();
src/java.security.jgss/share/classes/sun/security/krb5/KdcComm.java line 349:
> 347: DEBUG.println(">>> KDCCommunication: kdc=" + kdc
> 348: + " " + proto + ":"
> 349: + port + ", timeout="
Suggestion:
+ port + ", timeout="
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22256#discussion_r1851454369
PR Review Comment: https://git.openjdk.org/jdk/pull/22256#discussion_r1851453729
More information about the core-libs-dev
mailing list