Looks good Vyom.
- Michael
On 12/09/2017, 09:46, vyom tewari wrote:
On Tuesday 12 September 2017 02:12 PM, Alan Bateman wrote:
On 12/09/2017 09:06, 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.
Moving jdk.httpserver to the platform class loader looks fine. Are
you planning a second phase to identify the permissions needed so
that it doesn't have to be granted AllPermission?
yes, i will file a separate issue for this.
Vyom
-Alan