crypto: remove default encoding from scrypt

Refs: #47182 PR-URL: #47943 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · May 15, 2023 · d799347 · d799347
1 parent 6f5ba92
commit d799347
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/lib/internal/crypto/scrypt.js b/lib/internal/crypto/scrypt.js
@@ -28,7 +28,6 @@ const {
 
 const {
   getArrayBufferOrView,
-  getDefaultEncoding,
 } = require('internal/crypto/util');
 
 const defaults = {
@@ -53,14 +52,11 @@ function scrypt(password, salt, keylen, options, callback = defaults) {
   const job = new ScryptJob(
     kCryptoJobAsync, password, salt, N, r, p, maxmem, keylen);
 
-  const encoding = getDefaultEncoding();
   job.ondone = (error, result) => {
     if (error !== undefined)
       return FunctionPrototypeCall(callback, job, error);
     const buf = Buffer.from(result);
-    if (encoding === 'buffer')
-      return FunctionPrototypeCall(callback, job, null, buf);
-    FunctionPrototypeCall(callback, job, null, buf.toString(encoding));
+    return FunctionPrototypeCall(callback, job, null, buf);
   };
 
   job.run();
@@ -77,9 +73,7 @@ function scryptSync(password, salt, keylen, options = defaults) {
   if (err !== undefined)
     throw err;
 
-  const buf = Buffer.from(result);
-  const encoding = getDefaultEncoding();
-  return encoding === 'buffer' ? buf : buf.toString(encoding);
+  return Buffer.from(result);
 }
 
 function check(password, salt, keylen, options) {