<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Igor,<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 31 Aug 2018, at 19:42, Igor Ignatyev <<a href="mailto:igor.ignatyev@oracle.com" class="">igor.ignatyev@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html; charset=utf-8" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Alan, Chris,<div class=""><br class=""></div><div class="">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.</div></div></div></blockquote><div><br class=""></div><div>It’s bit annoying to have retrofit the test library classes with doPriv,</div><div>but I think that it is the right approach, if we intend them to be</div><div>usable with tests running with a security manager ( which we do ).</div><br class=""><blockquote type="cite" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div class="">the incremental webrev can found here[1], please let me know if you need the whole webrev.</div><div class=""><br class=""></div><div class="">[1] <a href="http://cr.openjdk.java.net/~iignatyev//8210039/webrev.0-1/index.html" class="">http://cr.openjdk.java.net/~iignatyev//8210039/webrev.0-1/index.html</a></div></div></blockquote><br class=""></div><div>I think this is fine.</div><div><br class=""></div><div>Thanks,</div><div>-Chris.</div></body></html>