This patch looks fine.  It's fine with me to follow up the second phase to identify the permissions needed rather than granting AllPermissions.

Mandy

On 9/12/17 1:06 AM, vyom tewari wrote:
Hi,

Please review the below code change.

BugId: https://bugs.openjdk.java.net/browse/JDK-8159526

Webrev-1: http://cr.openjdk.java.net/~vtewari/8159526/jdk/webrev/index.html

Webrev-2: http://cr.openjdk.java.net/~vtewari/8159526/root/webrev/index.html

Code change will De-privilege jdk.httpserver, we gave "jdk.httpserver" all permission for now.

Thanks,

Vyom


Reply via email to