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