On Tue, 19 Nov 2024 11:00:49 GMT, Michael McMahon <micha...@openjdk.org> wrote:
>> 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 three additional >> commits since the last revision: >> >> - update the specification of >> java.security.SecureClassLoader.getPermissions() >> - merge latest from master branch >> - 8343150: Remove URLClassLoader.getPermissions after JEP 486 integration > > src/java.base/share/classes/java/security/SecureClassLoader.java line 177: > >> 175: * >> 176: * @return the permissions for the codesource. >> 177: * > > This change will have to go in the CSR as well presumably. Hello Michael, right - I have been talking to Alan and Sean about this change and there has been some inputs from Joe in the CSR too. This latest change to the SecureClassLoader was one proposal that's being considered. I suspect we will need at least one more day to come to a consensus on how we want to proceed. But yes, if we do decide to make this change, I'll update the CSR text. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22179#discussion_r1848153148
