On Mon, 12 May 2025 14:35:45 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   comments on the 11th
>
> src/java.base/share/conf/security/java.security line 1560:
> 
>> 1558: # withEncryption method.
>> 1559: #
>> 1560: jdk.epkcs8.defaultAlgorithm=PBEWithHmacSHA256AndAES_128
> 
> What about naming this `jdk.pem.pbeAlgorithm`? The `withEncryption` method 
> never mentions PKCS#8, so maybe don't need to expose this in the property 
> name. Also having "pem" in the property name makes it more clear what API 
> this property is used for.

The security property is not just PEMEncoder/PEMDecoder, it also affects EKPI.  
So slightly less PEM specific.  I'm open to change it if we need to

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17543#discussion_r2088115825

Reply via email to