Skip to content

Conversation

@kkoyung
Copy link
Member

@kkoyung kkoyung commented Nov 28, 2025

Start adding Argon2 support to WebCrypto API.

This patch implements import key operation of Argon2.

Testing:

  • Pass some WPT tests that were expected to fail.
  • Some new FAIL expectations are added to argon2.tentative.https.any.js.ini, which require the not-yet-implemented derive bits operation of Argon2. These tests were skipped by WPT when the import key operation of Argon2 had not been implemented at the time.

Fixes: Part of #40687

Start adding Argon2 support to WebCrypto API.

This patch implements import key operation of Argon2.

Testing:
- Pass some WPT tests that were expected to fail.
- Some new FAIL expectations are added to
  `argon2.tentative.https.any.js.ini`, which require the
  not-yet-implemented derive bits operation of Argon2. These tests were
  skipped by WPT when the import key operation of Argon2 had not been
  implemented at the time.

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 06:16
@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
@jdm jdm 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 ceee556 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-import branch November 28, 2025 09:13
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