fix invalid invocation on getRandomValues()

cloudflare · Sep 17, 2024 · e54450b · e54450b
1 parent 563d72f
commit e54450b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/node/crypto.ts b/src/node/crypto.ts
@@ -4,8 +4,7 @@
 //
 import { ERR_METHOD_NOT_IMPLEMENTED } from 'node-internal:internal_errors';
 
-// eslint-disable-next-line @typescript-eslint/unbound-method
-export const getRandomValues = crypto.getRandomValues;
+export const getRandomValues = crypto.getRandomValues.bind(crypto);
 export const subtle = crypto.subtle;
 export const webcrypto = crypto;
 

diff --git a/src/workerd/api/node/tests/crypto_random-test.js b/src/workerd/api/node/tests/crypto_random-test.js
@@ -413,3 +413,13 @@ export const timingSafeEqualTest = {
     timingSafeEqual(new Uint8Array(1), new Uint8Array(1));
   },
 };
+
+// Ref: https://github.com/cloudflare/workerd/issues/2716
+export const getRandomValuesIllegalInvocation = {
+  async test() {
+    const crypto = await import('node:crypto');
+    const getRandomValues = crypto.getRandomValues;
+    strictEqual(getRandomValues(new Uint8Array(6)).length, 6);
+    strictEqual(crypto.getRandomValues(new Uint8Array(6)).length, 6);
+  },
+};