RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v28]
Kevin Driver
kdriver at openjdk.org
Mon May 13 22:15:07 UTC 2024
On Mon, 13 May 2024 14:12:17 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>>
>> re-enable preview annotations
>
> src/java.base/share/classes/sun/security/util/Debug.java line 142:
>
>> 140: System.err.println(" only dump output for the specified list");
>> 141: System.err.println(" of JCA engines. Supported values:");
>> 142: System.err.println(" Cipher, KDF, KeyAgreement, KeyGenerator,");
>
> Do we also need to add the option name itself? Somewhere neat line 100.
I'll keep this one open for now.
> test/jdk/com/sun/crypto/provider/KDF/TestHKDFInitialization.java line 1:
>
>> 1: /*
>
> Why the class name? Is this only about initialization?
I'll probably remove this test before the PR is merged.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1599146261
PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1599147522
More information about the security-dev
mailing list