RFR: 8340327: A common framework to support public key algorithms with standard parameter sets [v6]

Sean Mullan mullan at openjdk.org
Fri Oct 4 13:02:38 UTC 2024


On Thu, 3 Oct 2024 17:40:22 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> To prepare for new PQC algorithms like ML-KEM and ML-DSA where there are only named standardized parameter sets, a common framework is introduced.
>> 
>> A example of EdDSA implementation using this framework is included as a test.
>
> Weijun Wang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
> 
>  - Merge branch 'master' into 8340327
>  - more test, more RAW support, fix a bug on cleaning up getRawBytes output
>  - add support for private class RawKeySpec
>  - ensure key is intact after being used
>  - renames
>  - the fix

src/java.base/share/classes/sun/security/provider/NamedKeyFactory.java line 47:

> 45: import java.util.Objects;
> 46: 
> 47: /// An implementation extends this class to create its own `KeyFactory`.

Suggest saying something more here about what classes should use this, KeyFactory implementations that support encoding/decoding keys for algorithms that have named parameters, examples: ...

Same comment for the other Named SPI classes.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21167#discussion_r1787681877


More information about the security-dev mailing list