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.

Reply via email to