On Mon, 17 Jun 2024 15:59:37 GMT, Kevin Walls <kev...@openjdk.org> wrote:

>> JMX uses APIs related to the Security Mananger which are deprecated.  Use of 
>> AccessControlContext will be removed when Security Manager is removed.
>> 
>> Until then, updates are needed to not require setting  
>> -Djava.security.manager=allow to use JMX authentication.
>
> Kevin Walls has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove import

src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
 line 1306:

> 1304:                     return action.run();
> 1305:                 else
> 1306:                     return Subject.doAs(subject, action);

Code style comment: put braces around the if/else to be consistent with lines 
below.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19624#discussion_r1643390138

Reply via email to