On Fri, 15 Nov 2024 15:55:57 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

> Can I please get a review of this change which removes references to 
> `AccessController.doPriveleged()` and also removes a security check?
> 
> Specifically:
> 
> - The `AccessController.doPriveleged()` has been removed from the 
> implementation.
> - The overridden `getPermission()` method has been removed from this internal 
> implementation class.
> - The call to `URLClassPath.checkURL()` which used to invoke a 
> SecurityManager check in the presence of a SecurityManager has been removed.
> 
> No new test has been introduced and tier1 and tier2 testing is currently in 
> progress.

This pull request has now been integrated.

Changeset: da403887
Author:    Jaikiran Pai <j...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/da4038873622a19ec10130d617969892583073e1
Stats:     17 lines in 1 file changed: 0 ins; 14 del; 3 mod

8344315: Clean up sun.net.www.protocol.jrt.JavaRuntimeURLConnection after JEP 
486 integration

Reviewed-by: dfuchs, alanb

-------------

PR: https://git.openjdk.org/jdk/pull/22151

Reply via email to