From 72ee82813c12d4679c4a2bc246a8f9421f07853f Mon Sep 17 00:00:00 2001 From: Deokjin Kim Date: Fri, 1 Sep 2023 00:58:53 +0900 Subject: [PATCH] tls: refactor to use validateFunction Use validateFunction to remove duplicate implementation. Plus, remove `assert()` statement because validateFunction has the same purpose. --- lib/_tls_wrap.js | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js index 909f36dd00fe15..324096c282923e 100644 --- a/lib/_tls_wrap.js +++ b/lib/_tls_wrap.js @@ -831,10 +831,7 @@ TLSSocket.prototype._init = function(socket, wrap) { ssl.handshakes = 0; if (options.ALPNCallback) { - if (typeof options.ALPNCallback !== 'function') { - throw new ERR_INVALID_ARG_TYPE('options.ALPNCallback', 'Function', options.ALPNCallback); - } - assert(typeof options.ALPNCallback === 'function'); + validateFunction(options.ALPNCallback, 'options.ALPNCallback'); this[kALPNCallback] = options.ALPNCallback; ssl.ALPNCallback = callALPNCallback; ssl.enableALPNCb();