I think "monitor" permission sounds reasonable. Thanks for pointing it out!
/Staffan (mobile) > On 13 jan 2014, at 17:40, frederic parain <[email protected]> wrote: > > The code looks good to me (not an official reviewer). > > However, I'm surprised this command doesn't > require the java.lang.management.ManagementPermission("monitor") > when invoked from the DiagnosticCommandMBean. > Has this topic been discussed during the CCC review > or with the security team? > > Regards, > > Fred > >> On 13/01/2014 14:17, Fredrik Arvidsson wrote: >> Hi >> >> Please help me review the following small enhancement: >> >> Webrev: http://cr.openjdk.java.net/~farvidsson/8031304/webrev.00/ >> <http://cr.openjdk.java.net/%7Efarvidsson/8031304/webrev.00/> >> Jira: https://bugs.openjdk.java.net/browse/JDK-8031304 >> >> /Fredrik > > -- > Frederic Parain - Oracle > Grenoble Engineering Center - France > Phone: +33 4 76 18 81 17 > Email: [email protected]
