crypto: allow monkey patching of pseudoRandomBytes

Make `pseudoRandomBytes` and it's aliases `prng` and `rng` configurable to allow monkey patching. PR-URL: nodejs#24108 Refs: nodejs#22519 Refs: nodejs#23017 Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Ujjwal Sharma <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Vladimir de Turckheim <[email protected]> Reviewed-By: James M Snell <[email protected]>
Trott · Nov 25, 2018 · f85d636 · f85d636
1 parent 895fe2f
commit f85d636
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/crypto.js b/lib/crypto.js
@@ -247,19 +247,25 @@ Object.defineProperties(exports, {
   // The ecosystem needs those to exist for backwards compatibility.
   prng: {
     enumerable: false,
+    configurable: true,
+    writable: true,
     value: pendingDeprecation ?
       deprecate(randomBytes, 'crypto.prng is deprecated.', 'DEP0115') :
       randomBytes
   },
   pseudoRandomBytes: {
     enumerable: false,
+    configurable: true,
+    writable: true,
     value: pendingDeprecation ?
       deprecate(randomBytes,
                 'crypto.pseudoRandomBytes is deprecated.', 'DEP0115') :
       randomBytes
   },
   rng: {
     enumerable: false,
+    configurable: true,
+    writable: true,
     value: pendingDeprecation ?
       deprecate(randomBytes, 'crypto.rng is deprecated.', 'DEP0115') :
       randomBytes

diff --git a/test/parallel/test-crypto-random.js b/test/parallel/test-crypto-random.js
@@ -306,3 +306,12 @@ assert.throws(
     }
   );
 });
+
+
+['pseudoRandomBytes', 'prng', 'rng'].forEach((f) => {
+  const desc = Object.getOwnPropertyDescriptor(crypto, f);
+  assert.ok(desc);
+  assert.strictEqual(desc.configurable, true);
+  assert.strictEqual(desc.writable, true);
+  assert.strictEqual(desc.enumerable, false);
+});