test: cover webcrypto prototype pollution systematically · nodejs/node@2bf0de8 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 753
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 2bf0de8

Browse filesBrowse the repository at this point in the historyBrowse files

panva

authored and

aduh95

committed

test: cover webcrypto prototype pollution systematically

Drive the regression test from the WebCrypto algorithm registry so all supported algorithms and operations must add explicit coverage regardless of whether they are native-job backed or js-based. Signed-off-by: Filip Skokan <panva.ip@gmail.com> PR-URL: #63520 Reviewed-By: René <contact.9a5d6388@renegade334.me.uk> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent 1a4edb3 commit 2bf0de8
Copy full SHA for 2bf0de8

2 files changed

+796-363Lines changed: 796 additions & 363 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib/internal/crypto
  • util.js
• test/parallel
  • test-webcrypto-promise-prototype-pollution.mjs

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/internal/crypto/util.js‎

Copy file name to clipboardExpand all lines: lib/internal/crypto/util.js

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -960,6 +960,7 @@ module.exports = {
960	960	toBuf,
961	961
962	962	kNamedCurveAliases,
	963	+ kSupportedAlgorithms,
963	964	normalizeAlgorithm,
964	965	normalizeHashName,
965	966	hasAnyNotIn,

0 commit comments

Comments

0 (0)