Skip to content

Conversation

@kkoyung
Copy link
Member

@kkoyung kkoyung commented Nov 28, 2025

Finish adding Argon2 support to WebCrypto API, using the crate argon2 to support the cryptographic calculation.

This patch implements "derive bits" operation and the "get key length" operation of Argon2. Actual error messages are also provided for the existing "import key" operation of Argon2.

Testing: Pass some WPT tests that were expected to fail.
Fixes: Part of #40687

Finish adding Argon2 support to WebCrypto API, using the crate `argon2`
to support the cryptographic calculation.

This patch implements "derive bits" operation and the "get key length"
operation of Argon2. Actual error messages are also provided for the
existing "import key" operation of Argon2.

Testing: Pass some WPT tests that were expected to fail.
Fixes: Part of 40687

Signed-off-by: Kingsley Yung <kingsley@kkoyung.dev>
@kkoyung kkoyung requested a review from gterzian as a code owner November 28, 2025 09:57
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 28, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 28, 2025
@TimvdLippe TimvdLippe added this pull request to the merge queue Nov 28, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 28, 2025
Merged via the queue into servo:main with commit a15861c Nov 28, 2025
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 28, 2025
@kkoyung kkoyung deleted the argon2-derive-bits branch November 29, 2025 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants