On Wed, 2 Jun 2021 23:15:46 GMT, Bradford Wetmore <[email protected]> wrote:
>> The JceSecurityManager is currently a subclass of
>> java.security.SecurityManager. Now that JEP 411 has been integrated, this
>> class should be updated to no longer subclass SecurityManager.
>>
>> The only reason for using SecurityManager to easily get the Class Context
>> (call stack), but we can achieve the same effect by using the JDK 9 API
>> java.lang.StackWalkeer. None of the other SecurityManager API are used.
>>
>> I have run mach5 tier1/tier2 plus --test
>> jck:api/java_security,jck:api/javax_crypto,jck:api/javax_net,jck:api/javax_security,jck:api/org_ietf,jck:api/javax_xml/crypto
>> with all green.
>
> Bradford Wetmore has updated the pull request with a new target base due to a
> merge or a rebase. The pull request now contains eight commits:
>
> - Address codereview comments
> - Merge branch 'master' into JDK-8267485
> - Merge branch 'master' into JDK-8267485
> - Merge branch 'master' into JDK-8267485
> - Replace missing annotation
> - Merge branch 'master' into JDK-8267485
> - Updated copyright date.
> - 8267485: Remove the dependency on SecurityManager in
> JceSecurityManager.java
src/java.base/share/classes/javax/crypto/JceSecurityManager.java line 50:
> 48: * @since 1.4
> 49: */
> 50: @SuppressWarnings("removal")
You should remove this annotation now that the dependency on SecurityManager
has been removed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4150
