crypto: rename CShakeParams and KmacParams length to outputLength

PR-URL: #61875
Refs: web-platform-tests/wpt#57802
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>

Author: panva

doc/api/webcrypto.md

@@ -1869,19 +1869,27 @@ the message.
 
 <!-- YAML
 added: v24.7.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/61875
+    description: Renamed `cShakeParams.length` to `cShakeParams.outputLength`.
 -->
 
-#### `cShakeParams.customization`
+#### `cShakeParams.name`
 
 <!-- YAML
 added: v24.7.0
 -->
 
-* Type: {ArrayBuffer|TypedArray|DataView|Buffer|undefined}
+* Type: {string} Must be `'cSHAKE128'`[^modern-algos] or `'cSHAKE256'`[^modern-algos]
 
-The `customization` member represents the customization string.
-The Node.js Web Crypto API implementation only supports zero-length customization
-which is equivalent to not providing customization at all.
+#### `cShakeParams.outputLength`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+* Type: {number} represents the requested output length in bits.
 
 #### `cShakeParams.functionName`
 
@@ -1896,21 +1904,17 @@ functions based on cSHAKE.
 The Node.js Web Crypto API implementation only supports zero-length functionName
 which is equivalent to not providing functionName at all.
 
-#### `cShakeParams.length`
+#### `cShakeParams.customization`
 
 <!-- YAML
 added: v24.7.0
 -->
 
-* Type: {number} represents the requested output length in bits.
-
-#### `cShakeParams.name`
-
-<!-- YAML
-added: v24.7.0
--->
+* Type: {ArrayBuffer|TypedArray|DataView|Buffer|undefined}
 
-* Type: {string} Must be `'cSHAKE128'`[^modern-algos] or `'cSHAKE256'`[^modern-algos]
+The `customization` member represents the customization string.
+The Node.js Web Crypto API implementation only supports zero-length customization
+which is equivalent to not providing customization at all.
 
 ### Class: `EcdhKeyDeriveParams`
 
@@ -2387,6 +2391,10 @@ added: v24.8.0
 
 <!-- YAML
 added: v24.8.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/61875
+    description: Renamed `kmacParams.length` to `kmacParams.outputLength`.
 -->
 
 #### `kmacParams.algorithm`
@@ -2397,25 +2405,25 @@ added: v24.8.0
 
 * Type: {string} Must be `'KMAC128'` or `'KMAC256'`.
 
-#### `kmacParams.customization`
+#### `kmacParams.outputLength`
 
 <!-- YAML
-added: v24.8.0
+added: REPLACEME
 -->
 
-* Type: {ArrayBuffer|TypedArray|DataView|Buffer|undefined}
+* Type: {number}
 
-The `customization` member represents the optional customization string.
+The length of the output in bytes. This must be a positive integer.
 
-#### `kmacParams.length`
+#### `kmacParams.customization`
 
 <!-- YAML
 added: v24.8.0
 -->
 
-* Type: {number}
+* Type: {ArrayBuffer|TypedArray|DataView|Buffer|undefined}
 
-The length of the output in bytes. This must be a positive integer.
+The `customization` member represents the optional customization string.
 
 ### Class: `Pbkdf2Params`
 

lib/internal/crypto/hash.js

@@ -223,7 +223,7 @@ async function asyncDigest(algorithm, data) {
         kCryptoJobAsync,
         normalizeHashName(algorithm.name),
         data,
-        algorithm.length));
+        algorithm.outputLength));
   }
 
   throw lazyDOMException('Unrecognized algorithm name', 'NotSupportedError');

lib/internal/crypto/mac.js

@@ -234,7 +234,7 @@ function kmacSignVerify(key, data, algorithm, signature) {
     key[kKeyObject][kHandle],
     algorithm.name,
     algorithm.customization,
-    algorithm.length / 8,
+    algorithm.outputLength / 8,
     data,
     signature));
 }

lib/internal/crypto/webidl.js

@@ -591,14 +591,14 @@ converters.CShakeParams = createDictionaryConverter(
   'CShakeParams', [
     ...new SafeArrayIterator(dictAlgorithm),
     {
-      key: 'length',
+      key: 'outputLength',
       converter: (V, opts) =>
         converters['unsigned long'](V, { ...opts, enforceRange: true }),
       validator: (V, opts) => {
         // The Web Crypto spec allows for SHAKE output length that are not multiples of
         // 8. We don't.
         if (V % 8)
-          throw lazyDOMException('Unsupported CShakeParams length', 'NotSupportedError');
+          throw lazyDOMException('Unsupported CShakeParams outputLength', 'NotSupportedError');
       },
       required: true,
     },
@@ -881,13 +881,13 @@ converters.KmacParams = createDictionaryConverter(
   'KmacParams', [
     ...new SafeArrayIterator(dictAlgorithm),
     {
-      key: 'length',
+      key: 'outputLength',
       converter: (V, opts) =>
         converters['unsigned long'](V, { ...opts, enforceRange: true }),
       validator: (V, opts) => {
         // The Web Crypto spec allows for KMAC output length that are not multiples of 8. We don't.
         if (V % 8)
-          throw lazyDOMException('Unsupported KmacParams length', 'NotSupportedError');
+          throw lazyDOMException('Unsupported KmacParams outputLength', 'NotSupportedError');
       },
       required: true,
     },

test/fixtures/crypto/kmac.js

@@ -13,7 +13,7 @@ module.exports = function() {
       ]),
       data: Buffer.from([0x00, 0x01, 0x02, 0x03]),
       customization: undefined,
-      length: 256,
+      outputLength: 256,
       expected: Buffer.from([
         0xe5, 0x78, 0x0b, 0x0d, 0x3e, 0xa6, 0xf7, 0xd3, 0xa4, 0x29, 0xc5, 0x70,
         0x6a, 0xa4, 0x3a, 0x00, 0xfa, 0xdb, 0xd7, 0xd4, 0x96, 0x28, 0x83, 0x9e,
@@ -30,7 +30,7 @@ module.exports = function() {
       ]),
       data: Buffer.from([0x00, 0x01, 0x02, 0x03]),
       customization: Buffer.from('My Tagged Application'),
-      length: 256,
+      outputLength: 256,
       expected: Buffer.from([
         0x3b, 0x1f, 0xba, 0x96, 0x3c, 0xd8, 0xb0, 0xb5, 0x9e, 0x8c, 0x1a, 0x6d,
         0x71, 0x88, 0x8b, 0x71, 0x43, 0x65, 0x1a, 0xf8, 0xba, 0x0a, 0x70, 0x70,
@@ -47,7 +47,7 @@ module.exports = function() {
       ]),
       data: Buffer.from(Array.from({ length: 200 }, (_, i) => i)), // 0x00-0xC7
       customization: Buffer.from('My Tagged Application'),
-      length: 256,
+      outputLength: 256,
       expected: Buffer.from([
         0x1f, 0x5b, 0x4e, 0x6c, 0xca, 0x02, 0x20, 0x9e, 0x0d, 0xcb, 0x5c, 0xa6,
         0x35, 0xb8, 0x9a, 0x15, 0xe2, 0x71, 0xec, 0xc7, 0x60, 0x07, 0x1d, 0xfd,
@@ -64,7 +64,7 @@ module.exports = function() {
       ]),
       data: Buffer.from([0x00, 0x01, 0x02, 0x03]),
       customization: Buffer.from('My Tagged Application'),
-      length: 512,
+      outputLength: 512,
       expected: Buffer.from([
         0x20, 0xc5, 0x70, 0xc3, 0x13, 0x46, 0xf7, 0x03, 0xc9, 0xac, 0x36, 0xc6,
         0x1c, 0x03, 0xcb, 0x64, 0xc3, 0x97, 0x0d, 0x0c, 0xfc, 0x78, 0x7e, 0x9b,
@@ -84,7 +84,7 @@ module.exports = function() {
       ]),
       data: Buffer.from(Array.from({ length: 200 }, (_, i) => i)), // 0x00-0xC7
       customization: undefined,
-      length: 512,
+      outputLength: 512,
       expected: Buffer.from([
         0x75, 0x35, 0x8c, 0xf3, 0x9e, 0x41, 0x49, 0x4e, 0x94, 0x97, 0x07, 0x92,
         0x7c, 0xee, 0x0a, 0xf2, 0x0a, 0x3f, 0xf5, 0x53, 0x90, 0x4c, 0x86, 0xb0,
@@ -104,7 +104,7 @@ module.exports = function() {
       ]),
       data: Buffer.from(Array.from({ length: 200 }, (_, i) => i)), // 0x00-0xC7
       customization: Buffer.from('My Tagged Application'),
-      length: 512,
+      outputLength: 512,
       expected: Buffer.from([
         0xb5, 0x86, 0x18, 0xf7, 0x1f, 0x92, 0xe1, 0xd5, 0x6c, 0x1b, 0x8c, 0x55,
         0xdd, 0xd7, 0xcd, 0x18, 0x8b, 0x97, 0xb4, 0xca, 0x4d, 0x99, 0x83, 0x1e,

test/fixtures/webcrypto/supports-modern-algorithms.mjs

@@ -17,17 +17,17 @@ const X25519 = await subtle.generateKey('X25519', false, ['deriveBits', 'deriveK
 export const vectors = {
   'digest': [
     [false, 'cSHAKE128'],
-    [shake128, { name: 'cSHAKE128', length: 128 }],
-    [shake128, { name: 'cSHAKE128', length: 128, functionName: Buffer.alloc(0), customization: Buffer.alloc(0) }],
-    [false, { name: 'cSHAKE128', length: 128, functionName: Buffer.alloc(1) }],
-    [false, { name: 'cSHAKE128', length: 128, customization: Buffer.alloc(1) }],
-    [false, { name: 'cSHAKE128', length: 127 }],
+    [shake128, { name: 'cSHAKE128', outputLength: 128 }],
+    [shake128, { name: 'cSHAKE128', outputLength: 128, functionName: Buffer.alloc(0), customization: Buffer.alloc(0) }],
+    [false, { name: 'cSHAKE128', outputLength: 128, functionName: Buffer.alloc(1) }],
+    [false, { name: 'cSHAKE128', outputLength: 128, customization: Buffer.alloc(1) }],
+    [false, { name: 'cSHAKE128', outputLength: 127 }],
     [false, 'cSHAKE256'],
-    [shake256, { name: 'cSHAKE256', length: 256 }],
-    [shake256, { name: 'cSHAKE256', length: 256, functionName: Buffer.alloc(0), customization: Buffer.alloc(0) }],
-    [false, { name: 'cSHAKE256', length: 256, functionName: Buffer.alloc(1) }],
-    [false, { name: 'cSHAKE256', length: 256, customization: Buffer.alloc(1) }],
-    [false, { name: 'cSHAKE256', length: 255 }],
+    [shake256, { name: 'cSHAKE256', outputLength: 256 }],
+    [shake256, { name: 'cSHAKE256', outputLength: 256, functionName: Buffer.alloc(0), customization: Buffer.alloc(0) }],
+    [false, { name: 'cSHAKE256', outputLength: 256, functionName: Buffer.alloc(1) }],
+    [false, { name: 'cSHAKE256', outputLength: 256, customization: Buffer.alloc(1) }],
+    [false, { name: 'cSHAKE256', outputLength: 255 }],
   ],
   'sign': [
     [pqc, 'ML-DSA-44'],
@@ -44,8 +44,8 @@ export const vectors = {
     [false, 'Argon2id'],
     [false, 'KMAC128'],
     [false, 'KMAC256'],
-    [kmac, { name: 'KMAC128', length: 256 }],
-    [kmac, { name: 'KMAC256', length: 256 }],
+    [kmac, { name: 'KMAC128', outputLength: 256 }],
+    [kmac, { name: 'KMAC256', outputLength: 256 }],
   ],
   'generateKey': [
     [pqc, 'ML-DSA-44'],

test/fixtures/wpt/README.md

@@ -34,7 +34,7 @@ Last update:
 - wasm/jsapi: https://github.com/web-platform-tests/wpt/tree/cde25e7e3c/wasm/jsapi
 - wasm/webapi: https://github.com/web-platform-tests/wpt/tree/fd1b23eeaa/wasm/webapi
 - web-locks: https://github.com/web-platform-tests/wpt/tree/10a122a6bc/web-locks
-- WebCryptoAPI: https://github.com/web-platform-tests/wpt/tree/6a1c545d77/WebCryptoAPI
+- WebCryptoAPI: https://github.com/web-platform-tests/wpt/tree/0acea989ac/WebCryptoAPI
 - webidl: https://github.com/web-platform-tests/wpt/tree/63ca529a02/webidl
 - webidl/ecmascript-binding/es-exceptions: https://github.com/web-platform-tests/wpt/tree/2f96fa1996/webidl/ecmascript-binding/es-exceptions
 - webmessaging/broadcastchannel: https://github.com/web-platform-tests/wpt/tree/6495c91853/webmessaging/broadcastchannel

test/fixtures/wpt/WebCryptoAPI/digest/cshake.tentative.https.any.js

@@ -162,7 +162,7 @@ Object.keys(digestedData).forEach(function (alg) {
     Object.keys(sourceData).forEach(function (size) {
       promise_test(function (test) {
         return crypto.subtle
-          .digest({ name: alg, length: length }, sourceData[size])
+          .digest({ name: alg, outputLength: length }, sourceData[size])
           .then(function (result) {
             assert_true(
               equalBuffers(result, digestedData[alg][length][size]),
@@ -183,7 +183,7 @@ Object.keys(digestedData).forEach(function (alg) {
                 buffer[0] = sourceData[size][0];
                 return alg;
               },
-              length
+              outputLength: length
             }, buffer)
             .then(function (result) {
               assert_true(
@@ -196,7 +196,7 @@ Object.keys(digestedData).forEach(function (alg) {
         promise_test(function (test) {
           var buffer = new Uint8Array(sourceData[size]);
           var promise = crypto.subtle
-            .digest({ name: alg, length: length }, buffer)
+            .digest({ name: alg, outputLength: length }, buffer)
             .then(function (result) {
               assert_true(
                 equalBuffers(result, digestedData[alg][length][size]),
@@ -217,7 +217,7 @@ Object.keys(digestedData).forEach(function (alg) {
                 buffer.buffer.transfer();
                 return alg;
               },
-              length
+              outputLength: length
             }, buffer)
             .then(function (result) {
               assert_true(
@@ -230,7 +230,7 @@ Object.keys(digestedData).forEach(function (alg) {
         promise_test(function (test) {
           var buffer = new Uint8Array(sourceData[size]);
           var promise = crypto.subtle
-            .digest({ name: alg, length: length }, buffer)
+            .digest({ name: alg, outputLength: length }, buffer)
             .then(function (result) {
               assert_true(
                 equalBuffers(result, digestedData[alg][length][size]),