Skip to content

src: improve EC JWK import performance#62396

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
panva:improve-ec-jwk-import
Mar 29, 2026
Merged

src: improve EC JWK import performance#62396
nodejs-github-bot merged 1 commit intonodejs:mainfrom
panva:improve-ec-jwk-import

Conversation

@panva
Copy link
Copy Markdown
Member

@panva panva commented Mar 22, 2026

./node benchmark/crypto/create-keyobject.js keyFormat=jwk-public n=1000000 keyType=ec
Branch ops/sec
main 76,259
improve-ec-jwk-import 168,920

All EC JWK import supported curves go through the optimized branch but I left the old one in just in case.

@panva panva requested a review from jasnell March 22, 2026 19:05
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto
  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Mar 22, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@panva panva added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 28, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 29, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/62396
βœ”  Done loading data for nodejs/node/pull/62396
----------------------------------- PR info ------------------------------------
Title      src: improve EC JWK import performance (#62396)
Author     Filip Skokan <panva.ip@gmail.com> (@panva)
Branch     panva:improve-ec-jwk-import -> nodejs:main
Labels     author ready, needs-ci, dependencies
Commits    1
 - src: improve EC JWK import performance
Committers 1
 - Filip Skokan <panva.ip@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/62396
Reviewed-By: James M Snell <jasnell@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/62396
Reviewed-By: James M Snell <jasnell@gmail.com>
--------------------------------------------------------------------------------
   β„Ή  This PR was created on Sun, 22 Mar 2026 19:05:41 GMT
   βœ”  Approvals: 1
   βœ”  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/62396#pullrequestreview-4025611221
   ✘  This PR needs to wait 46 more minutes to land (or 0 minutes if there is one more approval)
   βœ”  Last GitHub CI successful
   β„Ή  Last Full PR CI on 2026-03-26T20:10:40Z: https://ci.nodejs.org/job/node-test-pull-request/72039/
- Querying data for job/node-test-pull-request/72039/
βœ”  Build data downloaded
   βœ”  Last Jenkins CI successful
--------------------------------------------------------------------------------
   βœ”  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/23715821406

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Mar 29, 2026
@panva panva added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Mar 29, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 29, 2026
@nodejs-github-bot nodejs-github-bot merged commit f3633ef into nodejs:main Mar 29, 2026
85 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in f3633ef

@panva panva deleted the improve-ec-jwk-import branch March 29, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants