Integrated: 8344188: Cleanup sun.net.www.protocol.jar.JarFileFactory after JEP 486 integration

Eirik Bjørsnøs eirbjo at openjdk.org
Fri Nov 15 12:21:55 UTC 2024


On Fri, 15 Nov 2024 07:56:15 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this PR which removes SM permission checking from `s.n.w.p.j.JarFileFactory`.
> 
> This removes the permission checking happening in `JarFileFactory::getCachedJarFile` and also removes the associated private `getPermission` method, (sadly including the cute `// gulp`). The import of `java.security.Permission` is also removed.
> 
> Testing: `test/jdk/sun/net/www/protocol/jar` runs green, GHA results pending.

This pull request has now been integrated.

Changeset: 5e276085
Author:    Eirik Bjørsnøs <eirbjo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/5e2760856c3844d9ad6200fef0c09ece0acac73a
Stats:     42 lines in 1 file changed: 0 ins; 41 del; 1 mod

8344188: Cleanup sun.net.www.protocol.jar.JarFileFactory after JEP 486 integration

Reviewed-by: jpai, dfuchs

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

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


More information about the net-dev mailing list