On Fri, 14 Jun 2024 12:44:17 GMT, Kevin Walls <kev...@openjdk.org> wrote:
>> 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.
>
> It will work to get the Subject yes.
> Do I not need the SM-enabled case, in case there some complex ACC in place?
> (combiner)
OK if it was actually getting specifically the ACC to use it maybe, but this
just needs to resolve a Subject.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19624#discussion_r1639806380
