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

Jaikiran Pai jpai at openjdk.org
Sat Nov 16 01:26:31 UTC 2024


On Fri, 15 Nov 2024 15:55:57 GMT, Jaikiran Pai <jpai at 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 <jpai at 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


More information about the net-dev mailing list