node icon indicating copy to clipboard operation
node copied to clipboard

crypto: fix webcrypto deriveBits validations

Open panva opened this issue 1 year ago • 2 comments

This PR fixes WebCryptoAPI deriveBits validation error behaviour.

Fixes 512 WPTs

panva avatar Aug 08 '22 10:08 panva

Review requested:

  • [ ] @nodejs/crypto

nodejs-github-bot avatar Aug 08 '22 10:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/45974/

nodejs-github-bot avatar Aug 10 '22 14:08 nodejs-github-bot

Commit Queue failed
- Loading data for nodejs/node/pull/44173
✔  Done loading data for nodejs/node/pull/44173
----------------------------------- PR info ------------------------------------
Title      crypto: fix webcrypto deriveBits validations (#44173)
Author     Filip Skokan  (@panva)
Branch     panva:fix-webcrypto-deriveBits-validations -> nodejs:main
Labels     crypto, experimental, author ready, needs-ci, review wanted, webcrypto
Commits    1
 - crypto: fix webcrypto deriveBits validations
Committers 1
 - Filip Skokan 
PR-URL: https://github.com/nodejs/node/pull/44173
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/44173
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Mon, 08 Aug 2022 10:40:48 GMT
   ✔  Approvals: 2
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/44173#pullrequestreview-1070984778
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/44173#pullrequestreview-1071674593
   ✖  Last GitHub CI failed
   ℹ  Last Full PR CI on 2022-08-10T14:41:34Z: https://ci.nodejs.org/job/node-test-pull-request/45974/
- Querying data for job/node-test-pull-request/45974/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/2848925395

nodejs-github-bot avatar Aug 12 '22 19:08 nodejs-github-bot

Landed in c6c4cdab361d

panva avatar Aug 12 '22 19:08 panva

THis needs backporting to v16.x

juanarbol avatar Sep 29 '22 21:09 juanarbol