Skip to content

[v24.x backport] crypto: remove Argon2 KDF derivation from its job setup#63170

Closed
panva wants to merge 1 commit intonodejs:v24.x-stagingfrom
panva:backport-62863-to-24
Closed

[v24.x backport] crypto: remove Argon2 KDF derivation from its job setup#63170
panva wants to merge 1 commit intonodejs:v24.x-stagingfrom
panva:backport-62863-to-24

Conversation

@panva
Copy link
Copy Markdown
Member

@panva panva commented May 7, 2026

Backport of #62861 for v24.x, the only change is the expected error message (which depended on #62348 semver-major PRs that contain breaking changes and should be released in the next major version. )

Fixes: nodejs#62861

Signed-off-by: Filip Skokan <panva.ip@gmail.com>
PR-URL: nodejs#62863
Fixes: nodejs#62861
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch. labels May 7, 2026
@panva
Copy link
Copy Markdown
Member Author

panva commented May 7, 2026

Will open 1 PR for multiple backports as requested.

@panva panva closed this May 7, 2026
@panva panva deleted the backport-62863-to-24 branch May 7, 2026 15:12
@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.07%. Comparing base (c989497) to head (0ffdb5e).
⚠️ Report is 478 commits behind head on v24.x-staging.

Additional details and impacted files
@@               Coverage Diff                @@
##           v24.x-staging   #63170     +/-   ##
================================================
  Coverage          90.06%   90.07%             
================================================
  Files                672      675      +3     
  Lines             201174   203742   +2568     
  Branches           39304    39910    +606     
================================================
+ Hits              181197   183520   +2323     
- Misses             12240    12427    +187     
- Partials            7737     7795     +58     
Files with missing lines Coverage Δ
src/crypto/crypto_argon2.cc 63.33% <ø> (-0.40%) ⬇️
src/node_errors.h 86.48% <ø> (ø)

... and 194 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants