On Thu, 9 Apr 2026 16:22:11 GMT, Sean Mullan <[email protected]> wrote:

>> +1. It would make sense to also document that that the `include` directive 
>> is not supported.
>> 
>> Suggestion:
>> 
>>     Override the security properties - if they exist - in the 
>> `conf/security/java.security`
>>     configuration file with the properties in the specified file. Appends 
>> properties not
>>     previously present in `java.security` at the end. The `include` 
>> directive is not supported.
>
> Agree that we should be more clear to avoid confusion. Thanks for this 
> suggested change. I may tweak it slightly but it looks good.

Fixed in 
https://github.com/openjdk/jdk/pull/30635/changes/8e6219ff475d9982aee571d1e1e799415e10ecf7.
 Working on adding the include directive as an option next.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30635#discussion_r3100653144

Reply via email to