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

Kevin Driver kdriver at openjdk.org
Thu Sep 5 22:41:23 UTC 2024


On Tue, 3 Sep 2024 20:36:08 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   change impl class to use byte arrays rather than SecretKey objects where possible
>
> test/jdk/com/sun/crypto/provider/KDF/BasicHKDFFunctions.java line 54:
> 
>> 52:         var okm2 = kdf.deriveKey("OKM", extractAndExpand);
>> 53: 
>> 54:         if (!Arrays.equals(prk.getEncoded(), expectedPrk)) {
> 
> You can use `Asserts.assertEqualsByteArray` in `/test/lib`.

See: https://github.com/openjdk/jdk/pull/20301/commits/59b1743fd225ff34e6bcce055fd47a887ed22a08.

> test/jdk/com/sun/crypto/provider/KDF/BasicHKDFFunctions.java line 67:
> 
>> 65:         test(HKDFParameterSpec.ofExtract().extractOnly());
>> 66:         // test expand with empty info
>> 67:         test(HKDFParameterSpec.ofExtract().thenExpand(new byte[0], 32));
> 
> Do you want to test `info == null` also?

See: https://github.com/openjdk/jdk/pull/20301/commits/59b1743fd225ff34e6bcce055fd47a887ed22a08.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1746249096
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1746248828


More information about the security-dev mailing list