RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v58]

Kevin Driver kdriver at openjdk.org
Tue Oct 15 23:21:37 UTC 2024


On Tue, 15 Oct 2024 22:55:17 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   running the Exhaustive test through a more ruthless formatter
>
> test/jdk/com/sun/crypto/provider/KDF/HKDFExhaustiveTest.java line 298:
> 
>> 296:     Utils.runAndCheckException(
>> 297:         () -> KDF.getInstance(KDF_ALGORITHMS[0], (KDFParameters) new KDFAlgorithmParameterSpec()),
>> 298:         ClassCastException.class);
> 
> The ClassCastException is the result of line 297? I don't see the reason for testing this, seems nothing to do with the KDF impls?

@rhalade : can you comment on your test?

> test/jdk/com/sun/crypto/provider/KDF/HKDFExhaustiveTest.java line 328:
> 
>> 326:                 (KDFParameters) new KDFAlgorithmParameterSpec(),
>> 327:                 SUNJCE_PROVIDER),
>> 328:         ClassCastException.class);
> 
> Same comment regarding the ClassCastException here.

@rhalade : can you comment on your test?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1802119561
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1802119773


More information about the security-dev mailing list