RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v3]

Sean Mullan mullan at openjdk.org
Wed Oct 30 20:16:48 UTC 2024


On Fri, 25 Oct 2024 20:44:25 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Sean Mullan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 150 commits:
>> 
>>  - Merge remote-tracking branch 'jdk-sandbox/jep486' into JDK-8338411
>>  - Merge
>>  - Update @summary to replace "if the right permission is granted" can be replaced with "package java.lang is open to unnamed module".
>>  - Remove println about Security Manager.
>>  - Remove unused static variable NEW_PROXY_IN_PKG.
>>  - Remove static variable `DEFAULT_POLICY` and unused imports.
>>  - Remove hasSM() method and code that calls it, and remove comment about
>>    running test manually with SM.
>>  - clientlibs: import order
>>  - warning-string
>>  - java/net/httpclient/websocket/security/WSURLPermissionTest.java: integrated review feedback in renamed WSSanityTest.java
>>  - ... and 140 more: https://git.openjdk.org/jdk/compare/f7a61fce...cb50dfde
>
> test/jdk/java/util/logging/TestAppletLoggerContext.java line 40:
> 
>> 38:  * @modules java.base/jdk.internal.access
>> 39:  *          java.logging
>> 40:  * @run main/othervm TestAppletLoggerContext LoadingApplet
> 
> Rename these?  What's really being tested, there are no more Applets.

Fixed in https://github.com/openjdk/jdk/pull/21498/commits/444fabe80a7b53ba208db99d69b9778a6113454d

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1823302577


More information about the nio-dev mailing list