tls: fix performance regression in convertALPNProtocols()

`isUint8Array()` covers instances of `Buffer` `isArrayBufferView()` path is cold and not worth additional check PR-URL: nodejs/node#43250 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
noslate-project · Oct 10, 2022 · 435ad02 · 435ad02
1 parent 9a541f2
commit 435ad02
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/lib/tls.js b/lib/tls.js
@@ -56,7 +56,6 @@ const internalUtil = require('internal/util');
 internalUtil.assertCrypto();
 const {
   isArrayBufferView,
-  isDataView,
   isUint8Array,
 } = require('internal/util/types');
 
@@ -149,16 +148,14 @@ exports.convertALPNProtocols = function convertALPNProtocols(protocols, out) {
   // If protocols is Array - translate it into buffer
   if (ArrayIsArray(protocols)) {
     out.ALPNProtocols = convertProtocols(protocols);
-  } else if (Buffer.isBuffer(protocols) || isUint8Array(protocols)) {
+  } else if (isUint8Array(protocols)) {
     // Copy new buffer not to be modified by user.
     out.ALPNProtocols = Buffer.from(protocols);
-  } else if (isDataView(protocols)) {
+  } else if (isArrayBufferView(protocols)) {
     out.ALPNProtocols = Buffer.from(protocols.buffer.slice(
       protocols.byteOffset,
       protocols.byteOffset + protocols.byteLength
     ));
-  } else if (isArrayBufferView(protocols)) {
-    out.ALPNProtocols = Buffer.from(protocols.slice().buffer);
   }
 };