RFR: 8344216: Remove calls to SecurityManager and and doPrivileged in java.net.Authenticator, java.net.CookieHandler, and java.net.ResponseCache after JEP 486 integration
Daniel Fuchs
dfuchs at openjdk.org
Fri Nov 15 10:36:10 UTC 2024
On Fri, 15 Nov 2024 09:00:29 GMT, Eirik Bjørsnøs <eirbjo at 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
More information about the net-dev
mailing list