RFR: 8244336: Restrict algorithms at JCE layer [v2]

Valerie Peng valeriep at openjdk.org
Wed Aug 6 03:03:08 UTC 2025


On Thu, 31 Jul 2025 18:20:39 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review comments from Sean and Tony.
>
> src/java.base/share/conf/security/java.security line 775:
> 
>> 773: 
>> 774: #
>> 775: # Algorithm restrictions for Java Crypto API services
> 
> Nit: missing period.

Given this is a title, we don't need a period? This is same as other titles in the `java.security` file.

> src/java.base/share/conf/security/java.security line 789:
> 
>> 787: #       Service.AlgorithmName
>> 788: #
>> 789: # Service:  (one of the following, more service may be added later)
> 
> s/service/services/

Will fix.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26377#discussion_r2255747128
PR Review Comment: https://git.openjdk.org/jdk/pull/26377#discussion_r2255747624


More information about the security-dev mailing list