On Thu, 3 Oct 2024 17:40:22 GMT, Weijun Wang <wei...@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