RFR: 8331008: Implement JEP 478: Key Derivation Function API (Preview) [v36]
Kevin Driver
kdriver at openjdk.org
Mon Sep 16 16:39:30 UTC 2024
On Sat, 14 Sep 2024 22:11:21 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>>
>> refine wording on DPS getInstance with params exception
>
> src/java.base/share/classes/javax/crypto/KDF.java line 194:
>
>> 192: * otherwise {@code null} is returned.
>> 193: *
>> 194: * @see <a href="#DelayedProviderSelection">Delayed Provider Selection</a>
>
> Why `@see DPS`? The paragraph above has nothing to do with it.
>
> If you meant the warning not to call it too early (which I think is not necessary), why not add the same to the `getProviderName` method?
`@see DPS` is also present in the header/javadoc for `getProviderName` currently. I don't think I understand that part of your comment.
The intent is to refer the reader to the *entire* DPS selection of the class specification when calling a method which is affected by DPS.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20301#discussion_r1761480754
More information about the security-dev
mailing list