tls: tweak clientCertEngine argument parsing

nodejs · Jun 2, 2021 · 954ea8c · 954ea8c
1 parent def5758
commit 954ea8c
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/internal/tls.js b/lib/internal/tls.js
@@ -305,15 +305,15 @@ function configSecureContext(context, options = {}, name = 'options') {
     }
   }
 
-  if (clientCertEngine !== undefined) {
+  if (typeof clientCertEngine === 'string') {
     if (typeof context.setClientCertEngine !== 'function')
       throw new ERR_CRYPTO_CUSTOM_ENGINE_NOT_SUPPORTED();
-    if (typeof clientCertEngine !== 'string') {
-      throw new ERR_INVALID_ARG_TYPE(`${name}.clientCertEngine`,
+    else
+      context.setClientCertEngine(clientCertEngine);
+  } else if (clientCertEngine !== undefined) {
+    throw new ERR_INVALID_ARG_TYPE(`${name}.clientCertEngine`,
                                      ['string', 'null', 'undefined'],
                                      clientCertEngine);
-    }
-    context.setClientCertEngine(clientCertEngine);
   }
 
   if (ticketKeys !== undefined) {