On Fri, 15 Nov 2024 09:00:29 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:

> Please review this PR to clean up SM use in `java.net.Authenticator`, 
> `java.net.CookieHandler`, and `java.net.ResponseCache` after JEP 486 
> integration.
> 
> * `Authenticator` is updated to remove calls to SM::checkPermission`
> * `CookieHandler` is updated to remove calls to SM::checkPermission` and 
> remove the import of `SecurityConstants`
> * `ResponseCache` is updated to remove calls to SM::checkPermission` and 
> remove the import of `SecurityConstants`
> * `SecurityConstants` is updated to remove four now-unused permission 
> constants
> 
> Testing: Tests in `test/jdk/java/net/Authenticator`, 
> `test/jdk/java/net/HttpCookie` and `test/jdk/java/net/ResponseCache` run 
> green. GHA results pending.

Changes look good. Please run tier2 to validate.

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

PR Review: https://git.openjdk.org/jdk/pull/22139#pullrequestreview-2438245003

Reply via email to