On Fri, 11 Sep 2026 17:48:32 GMT, Sean Mullan <[email protected]> wrote:
>> Alternate implementation - this version only contains one argument - the >> name of the properties file. If an include statement is needed, it should be >> inserted in the properties file and it will always be added as the last line >> of the the conf/security/java.security configuration file.. >> >> See https://github.com/openjdk/jdk/pull/30635 for the other implementation. >> >> This is a new jlink plugin which allows the user to specify values of >> security properties it wants to override in the conf/security/java.security >> configuration file in a custom runtime image. This enhancement, along with >> https://github.com/openjdk/jdk/pull/29700 allow users to more easily create >> runtimes that address the specific security requirements of their >> applications. >> >> The command-line syntax takes a file containing properties that the user >> wants to override. The file can also contain an include statement which will >> be added as the last line of the conf/security/java.security configuration >> file. >> >> For example: >> >> jlink --security-properties props.security >> >> where props.security is a file containing one more more properties in the >> java.security file syntax. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Sean Mullan has updated the pull request incrementally with eight additional > commits since the last revision: > > - Remove whitespace, fix continuation char. > - Test property with EOF immediately after continuation character > - Disallow include statement in java.security file. > - Check for duplicates in java.security file. > - Remove redundant prop. > - Remove extra back-tick. > - Test more properties with different escape characters. > - Store properties as specified by Properties.store(OutputStream, String). I have posted an update with the following changes: - Properties are now stored according to the `Properties.store(OutputStream, String)` specification. I copied an internal method from the implementation to make sure it was consistent. - Duplicate properties and include statements in the original `java.security` file are treated as parsing errors. These should be rare, if any usages, but if someone thinks they should be supported, I am willing to hear your rationale. - Several other smaller fixes and additional test cases ------------- PR Comment: https://git.openjdk.org/jdk/pull/31884#issuecomment-5639040671
