On Thu, 28 Nov 2024 13:58:58 GMT, Aleksei Efimov <aefi...@openjdk.org> 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` JNDI 
>> environment property can be used to setup the `SecurityManager`: if the 
>> property is added to an environment map the `RegistryContext` attempts to 
>> setup the `SecurityManager`. The environment property and related method are 
>> removed in this PR. 
>> 
>> `tier1` - `tier3`, and other non-tiered JNDI tests showed no failures 
>> related to this change.
>
> Aleksei Efimov has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove com.sun.jndi.ldap.VersionHelper

LGTM

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

PR Review: https://git.openjdk.org/jdk/pull/22421#pullrequestreview-2468568335

Reply via email to