RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v15]
Kevin Driver
kdriver at openjdk.org
Mon Aug 26 18:19:14 UTC 2024
On Mon, 26 Aug 2024 17:43:31 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>>
>> code review comments and test renaming
>
> src/java.base/share/classes/javax/crypto/KDF.java line 317:
>
>> 315: * if no {@code Provider} supports a {@code KDFSpi} implementation for
>> 316: * the specified algorithm
>> 317: * @throws InvalidAlgorithmParameterException
>
> No IAPE is thrown in the current implementation.
This is declared for implementation use. Are you arguing that no implementation may require it?
> src/java.base/share/classes/javax/crypto/KDF.java line 473:
>
>> 471: * @param alg
>> 472: * the algorithm of the resultant {@code SecretKey} object
>> 473: * @param derivationParameterSpec
>
> I prefer a short name like `input`. I know the type is `AlgorithmParameterSpec` but it's actually not parameter.
Acknowledged. `input` is too vague.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1731628819
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1731629746
More information about the security-dev
mailing list