> Can I please get a review of this change which proposes to remove > SecurityManager related API usages from `URLClassLoader` and its related > `URLClassPath`? This addresses https://bugs.openjdk.org/browse/JDK-8344223. > > The `URLClassLoader.getPermissions()` method will need additional changes but > that will be done as part of https://bugs.openjdk.org/browse/JDK-8343150. > > I'll be opening a CSR for this change because we are removing support for the > `jdk.net.URLClassPath.disableRestrictedPermissions` system property. > > No new tests have been added and existing tier1, tier2 and tier3 tests > continue to pass.
Jaikiran Pai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - merge latest from master branch - 8344223: Remove calls to SecurityManager and doPrivileged in java.net.URLClassLoader after JEP 486 integration ------------- Changes: - all: https://git.openjdk.org/jdk/pull/22233/files - new: https://git.openjdk.org/jdk/pull/22233/files/d4dff489..5a82b729 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=22233&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22233&range=00-01 Stats: 964 lines in 29 files changed: 466 ins; 419 del; 79 mod Patch: https://git.openjdk.org/jdk/pull/22233.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/22233/head:pull/22233 PR: https://git.openjdk.org/jdk/pull/22233
