RFR: 8344397: Remove Security Manager dependencies from java.security and sun.security packages [v5]

Sean Mullan mullan at openjdk.org
Mon Dec 2 18:12:55 UTC 2024


> Now that JEP 486 has been integrated, `java.security` and `sun.security` implementation dependencies on `System.getSecurityManager` and `AccessController.doPrivileged` can be removed. 
> 
> This should cover most of the remaining cleanup in the security libraries area. A couple of the `sun.security.action` classes can't be removed yet, as well as a few constants in `sun.security.util.SecurityConstants` and internal `ProtectionDomain` methods until dependencies are removed in other areas. Some additional test cleanup can also be done later.

Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:

  Remove PutAllAction and GetBooleanAction classes as there are no more dependencies.
  Move GetPropertyAction.privilegedGetTimeoutProp and privilegedGetBooleanProp
  methods to sun.security.util.SecurityProperties and rename them.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22418/files
  - new: https://git.openjdk.org/jdk/pull/22418/files/a447dd44..fdc7dfd3

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22418&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22418&range=03-04

  Stats: 365 lines in 7 files changed: 100 ins; 260 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/22418.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22418/head:pull/22418

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


More information about the security-dev mailing list