Igor, > On 31 Aug 2018, at 19:42, Igor Ignatyev <igor.ignat...@oracle.com> wrote: > > Alan, Chris, > > thanks for looking at it, I went w/ the alternative suggested by Chris. that > required a sprinkle of doPrivileged in the testlibrary, but now > Sockets/policy.* files grant the minimal required permissions to the test > code.
It’s bit annoying to have retrofit the test library classes with doPriv, but I think that it is the right approach, if we intend them to be usable with tests running with a security manager ( which we do ). > the incremental webrev can found here[1], please let me know if you need the > whole webrev. > > [1] http://cr.openjdk.java.net/~iignatyev//8210039/webrev.0-1/index.html > <http://cr.openjdk.java.net/~iignatyev//8210039/webrev.0-1/index.html> I think this is fine. Thanks, -Chris.