Re: RFR: 8344299: SM cleanup in javax.naming modules [v3]

2024-11-28 Thread Daniel Fuchs
On Thu, 28 Nov 2024 17:28:59 GMT, Aleksei Efimov wrote: >> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and >> `AccessController` usages from `java.naming`, `jdk.naming.rmi` and >> `jdk.naming.dns` modules. >> >> One noteworthy change: The `java.naming.rmi.security.manager

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Aleksei Efimov
On Thu, 28 Nov 2024 17:03:13 GMT, Alan Bateman wrote: > Sean has hollowed out SecurityConstants in pull/22418 but had to leave > GET_CLASSLOADER_PERMISSION because of this usage. There will be many rounds > of cleanup so doesn't matter if it comes later. ALL_PERMISSION will probably > be the l

Re: RFR: 8344299: SM cleanup in javax.naming modules [v3]

2024-11-28 Thread Alan Bateman
On Thu, 28 Nov 2024 17:26:04 GMT, Aleksei Efimov wrote: >> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and >> `AccessController` usages from `java.naming`, `jdk.naming.rmi` and >> `jdk.naming.dns` modules. >> >> One noteworthy change: The `java.naming.rmi.security.manager

Re: RFR: 8344299: SM cleanup in javax.naming modules [v3]

2024-11-28 Thread Aleksei Efimov
> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and > `AccessController` usages from `java.naming`, `jdk.naming.rmi` and > `jdk.naming.dns` modules. > > One noteworthy change: The `java.naming.rmi.security.manager` JNDI > environment property can be used to setup the `Securi

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Alan Bateman
On Thu, 28 Nov 2024 13:58:58 GMT, Aleksei Efimov wrote: >> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and >> `AccessController` usages from `java.naming`, `jdk.naming.rmi` and >> `jdk.naming.dns` modules. >> >> One noteworthy change: The `java.naming.rmi.security.manager

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Daniel Fuchs
On Thu, 28 Nov 2024 16:30:36 GMT, Alan Bateman wrote: >> Aleksei Efimov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove com.sun.jndi.ldap.VersionHelper > > src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.jav

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Daniel Fuchs
On Thu, 28 Nov 2024 13:58:58 GMT, Aleksei Efimov wrote: >> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and >> `AccessController` usages from `java.naming`, `jdk.naming.rmi` and >> `jdk.naming.dns` modules. >> >> One noteworthy change: The `java.naming.rmi.security.manager

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Alan Bateman
On Thu, 28 Nov 2024 13:58:58 GMT, Aleksei Efimov wrote: >> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and >> `AccessController` usages from `java.naming`, `jdk.naming.rmi` and >> `jdk.naming.dns` modules. >> >> One noteworthy change: The `java.naming.rmi.security.manager

Re: RFR: 8344299: SM cleanup in javax.naming modules

2024-11-28 Thread Aleksei Efimov
On Thu, 28 Nov 2024 07:52:14 GMT, Alan Bateman wrote: > Good cleanup. Maybe for a follow on PR but I assume VersionHelper can be > removed. There's been several post JEP 486 removals that mean that > createThread and loadClass shouldn't be needed now. Also isSerialDataAllowed > could move to O

Re: RFR: 8344299: SM cleanup in javax.naming modules [v2]

2024-11-28 Thread Aleksei Efimov
> The proposed change cleans-up `SecurityManager`, `doPriviledged`, and > `AccessController` usages from `java.naming`, `jdk.naming.rmi` and > `jdk.naming.dns` modules. > > One noteworthy change: The `java.naming.rmi.security.manager` JNDI > environment property can be used to setup the `Securi

Re: RFR: 8344299: SM cleanup in javax.naming modules

2024-11-27 Thread Alan Bateman
On Thu, 28 Nov 2024 00:17:43 GMT, Aleksei Efimov wrote: > The proposed change cleans-up `SecurityManager`, `doPriviledged`, and > `AccessController` usages from `java.naming`, `jdk.naming.rmi` and > `jdk.naming.dns` modules. > > One noteworthy change: The `java.naming.rmi.security.manager` JND

RFR: 8344299: SM cleanup in javax.naming modules

2024-11-27 Thread Aleksei Efimov
The proposed change cleans-up `SecurityManager`, `doPriviledged`, and `AccessController` usages from `java.naming`, `jdk.naming.rmi` and `jdk.naming.dns` modules. One noteworthy change: The `java.naming.rmi.security.manager` JNDI environment property can be used to setup the `SecurityManager`: