RFR: 8353001: Remove leftover Security Manager parsing code in sun.security.util.Debug

Sean Mullan mullan at openjdk.org
Tue Apr 29 19:25:45 UTC 2025


On Thu, 17 Apr 2025 17:32:03 GMT, Koushik Muthukrishnan Thirupattur <duke at openjdk.org> wrote:

> The private marshal() method in sun.security.util.Debug still contains code to parse "permission=<classname>" and "codebase=<URL>" options. These sub-options were part of the "access" option which was removed in JDK 24 as part of JEP 486, so this code can be removed.

src/java.base/share/classes/sun/security/util/Debug.java line 359:

> 357: 
> 358:             // convert to lower-case characters
> 359:             return String.valueOf(args.toLowerCase(Locale.ENGLISH));

You can just return `args.toLowerCase(Locale.ENGLISH)`. Or better yet, you don't need the `marshal` method anymore, and can just replace the one call to marshal with this line.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24731#discussion_r2067191328


More information about the security-dev mailing list