On Fri, 14 Jun 2024 12:04:06 GMT, Weijun Wang <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Separate SM allowed and not allowed cases
>
> src/java.management/share/classes/com/sun/jmx/remote/internal/ServerNotifForwarder.java
> line 353:
>
>> 351: } else {
>> 352: return Subject.getSubject(AccessController.getContext());
>> 353: }
>
> `Subject.current()` should work for both cases. See the impl of it.
ok!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19624#discussion_r1639766973
