RFR: 8344216: Remove calls to SecurityManager and and doPrivileged in java.net.Authenticator, java.net.CookieHandler, and java.net.ResponseCache after JEP 486 integration

Eirik Bjørsnøs eirbjo at openjdk.org
Fri Nov 15 09:11:59 UTC 2024


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.

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

Commit messages:
 - Remove SecurityManager permission checking from Authenticator, CookieHandler, ResponseCache

Changes: https://git.openjdk.org/jdk/pull/22139/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22139&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8344216
  Stats: 86 lines in 4 files changed: 0 ins; 85 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22139.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22139/head:pull/22139

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


More information about the net-dev mailing list