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.

This pull request has now been integrated.

Changeset: 73fd8919
Author:    Eirik Bjørsnøs <eir...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/73fd8919f581cd79d285d7f30bfac3457ef1127d
Stats:     86 lines in 4 files changed: 0 ins; 85 del; 1 mod

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

Reviewed-by: dfuchs

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

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

Reply via email to