Skip to content

[v24.x backport] raw key formats support in KeyObject APIs#62455

Open
panva wants to merge 2 commits intonodejs:v24.x-stagingfrom
panva:backport-62240-to-v24.x
Open

[v24.x backport] raw key formats support in KeyObject APIs#62455
panva wants to merge 2 commits intonodejs:v24.x-stagingfrom
panva:backport-62240-to-v24.x

Conversation

@panva
Copy link
Copy Markdown
Member

@panva panva commented Mar 27, 2026

Backports #62240 (partially) and #62480 to v24.x

The omitted portion of #62240 is the removal and cleanup of KeyExportJob use in WebCryptoAPI.

@panva panva added crypto Issues and PRs related to the crypto subsystem. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch. labels Mar 27, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto
  • @nodejs/performance

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Mar 27, 2026
@panva panva force-pushed the backport-62240-to-v24.x branch from 5c4faaa to 343e82c Compare March 27, 2026 09:32
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 80.39702% with 79 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.09%. Comparing base (065c9b0) to head (bd10385).
⚠️ Report is 159 commits behind head on v24.x-staging.

Files with missing lines Patch % Lines
src/crypto/crypto_keys.cc 68.75% 21 Missing and 49 partials ⚠️
lib/internal/crypto/keys.js 94.97% 8 Missing and 1 partial ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           v24.x-staging   #62455      +/-   ##
=================================================
- Coverage          90.10%   90.09%   -0.02%     
=================================================
  Files                667      670       +3     
  Lines             199019   200740    +1721     
  Branches           38911    39223     +312     
=================================================
+ Hits              179335   180863    +1528     
- Misses             12040    12162     +122     
- Partials            7644     7715      +71     
Files with missing lines Coverage Δ
src/crypto/crypto_keys.h 56.60% <ø> (+1.50%) ⬆️
src/node_errors.h 86.48% <ø> (+1.63%) ⬆️
lib/internal/crypto/keys.js 95.46% <94.97%> (+0.17%) ⬆️
src/crypto/crypto_keys.cc 72.54% <68.75%> (-1.42%) ⬇️

... and 172 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.

@nodejs-github-bot

This comment was marked as outdated.

@panva panva force-pushed the backport-62240-to-v24.x branch from 343e82c to d0ecd5f Compare March 27, 2026 12:36
@nodejs-github-bot

This comment was marked as outdated.

PR-URL: nodejs#62240
Reviewed-By: James M Snell <jasnell@gmail.com>
@panva panva force-pushed the backport-62240-to-v24.x branch from d0ecd5f to 8db948b Compare March 27, 2026 14:27
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@panva panva marked this pull request as draft March 28, 2026 18:13
@panva panva changed the title [v24.x backport] crypto: add raw key formats support to the KeyObject APIs [v24.x backport] raw key formats support in KeyObject APIs Mar 28, 2026
Signed-off-by: Filip Skokan <panva.ip@gmail.com>
PR-URL: nodejs#62480
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@panva panva marked this pull request as ready for review March 31, 2026 08:40
@panva panva added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Mar 31, 2026
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++. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. crypto Issues and PRs related to the crypto subsystem. lib / src Issues and PRs related to general changes in the lib or src directory. 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