1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

crypto: remove obsolete encoding check

Browse Source 
This renames the parameters for clarity and removes the check for
undefined encoding. That will always default to `utf8`.

PR-URL: https://github.com/nodejs/node/pull/26809
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Ruben Bridgewater 2019-03-20 13:05:12 +01:00
parent 92db780d9e
commit 751c92d972
No known key found for this signature in database
GPG Key ID: F07496B3EB3C1762
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/internal/crypto/util.js
View File

@ -55,13 +55,13 @@ function getDefaultEncoding() {
// This is here because many functions accepted binary strings without // This is here because many functions accepted binary strings without
// any explicit encoding in older versions of node, and we don't want // any explicit encoding in older versions of node, and we don't want
// to break them unnecessarily. // to break them unnecessarily.
function toBuf(str, encoding) { function toBuf(val, encoding) {
if (typeof str === 'string') { if (typeof val === 'string') {
if (encoding === 'buffer' || !encoding) if (encoding === 'buffer')
encoding = 'utf8'; encoding = 'utf8';
return Buffer.from(str, encoding); return Buffer.from(val, encoding);
} }
return str; return val;
} }
const getCiphers = cachedResult(() => filterDuplicateStrings(_getCiphers())); const getCiphers = cachedResult(() => filterDuplicateStrings(_getCiphers()));