RFR: 8346129: Simplify EdDSA & XDH curve name usage [v6]

Anthony Scarpino ascarpino at openjdk.org
Tue Mar 25 16:36:30 UTC 2025


On Tue, 25 Mar 2025 16:28:35 GMT, Artur Barashev <abarashev at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   comments on test
>
> test/jdk/sun/security/util/AlgorithmConstraints/DisabledAlgorithmPermits.java line 143:
> 
>> 141:                 throw new AssertionError("failed.  Expected " +
>> 142:                     tc.expected);
>> 143:             }
> 
> I suggest replacing this check with `assertEquals`.

If I did that, then I couldn't println to stdout.  I'd rather keep it as is.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23647#discussion_r2012498792


More information about the security-dev mailing list