tls: refactor to use validateFunction

Use validateFunction to remove duplicate implementation. Plus, remove `assert()` statement because validateFunction has the same purpose. PR-URL: #49422 Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Debadree Chatterjee <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Sep 10, 2023 · 83fc4dc · 83fc4dc
1 parent ca13841
commit 83fc4dc
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
@@ -834,10 +834,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();