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