Skip to content

[v16.x backport] crypto: fix webcrypto EC key namedCurve validation errors - #44837

Closed
panva wants to merge 1 commit into
nodejs:v16.x-stagingfrom
panva:backport-44172-to-v16.x
Closed

[v16.x backport] crypto: fix webcrypto EC key namedCurve validation errors#44837
panva wants to merge 1 commit into
nodejs:v16.x-stagingfrom
panva:backport-44172-to-v16.x

Conversation

@panva

@panvapanva commented Sep 30, 2022

Copy link
Copy Markdown
Member

Refs: #44172

This backports the runtime functionality without the portion that updated the wpt status file which in turn depended on the WPT Runner update (#43455).

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto

@nodejs-github-botnodejs-github-bot added crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. v16.x labels Sep 30, 2022
PR-URL: nodejs#44172
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Backport-PR-URL: nodejs#44837
@panva
panvaforce-pushed the backport-44172-to-v16.x branch from f3db251 to 878bd13CompareSeptember 30, 2022 12:04
@nodejs-github-bot

This comment was marked as outdated.

@panva

Copy link
Copy Markdown
MemberAuthor

cc @juanarbol this is the best I can do without having to backport the WPT runner update

@panvapanva added webcrypto request-ci Add this label to start a Jenkins CI on a PR. labels Sep 30, 2022
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 30, 2022
@nodejs-github-bot

This comment was marked as outdated.

@panvapanva added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 2, 2022
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 2, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

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

Labels

cryptoIssues and PRs related to the crypto subsystem.needs-ciPRs that need a full CI run.webcrypto

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@panva@nodejs-github-bot