On Sat, 17 Jun 2023 01:51:36 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Pass PrintStream to security helper
>
> src/java.base/share/classes/sun/launcher/SecuritySettings.java line 93:
>
>> 91: for (String key :
>> p.stringPropertyNames().stream().sorted().toList()) {
>> 92: String val = p.getProperty(key);
>> 93: if (val.contains(",") && val.length() > 60) {
>
> Do you want to do the same for `;`? `jceks.key.serialFilter` uses it.
fair point. done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14394#discussion_r1234252148
