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

Reply via email to