On Thu, 15 Oct 2026 19:54:42 GMT, Sean Mullan <[email protected]> wrote:

>> Hi @seanjmullan, thank you for the updates, I discovered other potential 
>> problems with the escaping of user-defined properties.
>
> @franferrax The CSR has been approved for this change and I have Alan's 
> approval to integrate. I believe I have resolved all of your comments. I plan 
> to integrate early next week. But let me know you need more time to review 
> the latest updates. Thanks.

Hi @seanjmullan, sorry for my delayed reply, I've been busy with the OJVG work. 
Glad to see you didn't bock on me. I think you achieved a good-enough level of 
alignment with the properties parsing.

Just to validate my understanding, if someone builds its own JDK with an 
`include` statement in `java.security` (not the _Red Hat_ case), the _Security 
properties jlink plugin_ will always fail, due to this check, right?
https://github.com/openjdk/jdk/blob/776ffc524e5ea825b73dd7e6b315fbc33765010b/src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/SecurityPropertiesPlugin.java#L157-L160

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

PR Comment: https://git.openjdk.org/jdk/pull/31884#issuecomment-6270395667

Reply via email to