On Wed, 25 Mar 2026 19:26:42 GMT, Sean Mullan <[email protected]> wrote:

>> Weijun Wang has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains six commits:
>> 
>>  - Merge branch 'master' into 8368692
>>  - Merge branch 'master' into 8368692
>>  - merge
>>  - more descriptive exception message
>>  - adressing Naoto's comments; check invalid property value
>>  - the change
>
> src/java.base/share/conf/security/java.security line 1751:
> 
>> 1749: # the JAAS KeyStoreLoginModule allows a password to be read from 
>> users-specified
>> 1750: # "keyStorePasswordURL". The keytool and jarsigner commands also 
>> support options
>> 1751: # like "-storepass:env" and "-storepass:file" to read passwords from an
> 
> Do you need to say "like"? if these are the only options, just remove "like".

There are also `-keypass:env` and `-newpass:env` etc. That's why I said "like".

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29490#discussion_r2991930919

Reply via email to