tls: rename validateKeyCert in _tls_common.js

This commit renames validateKeyCert to validateKeyCertArg to avoid
confusing this with something that would validate the actual key or
certificate.

Author: danbev

lib/_tls_common.js

@@ -75,7 +75,7 @@ function SecureContext(secureProtocol, secureOptions, minVersion, maxVersion) {
   if (secureOptions) this.context.setOptions(secureOptions);
 }
 
-function validateKeyCert(name, value) {
+function validateKeyCertArg(name, value) {
   if (typeof value !== 'string' && !isArrayBufferView(value)) {
     throw new ERR_INVALID_ARG_TYPE(
       `options.${name}`,
@@ -107,11 +107,11 @@ exports.createSecureContext = function createSecureContext(options) {
     if (Array.isArray(ca)) {
       for (i = 0; i < ca.length; ++i) {
         val = ca[i];
-        validateKeyCert('ca', val);
+        validateKeyCertArg('ca', val);
         c.context.addCACert(val);
       }
     } else {
-      validateKeyCert('ca', ca);
+      validateKeyCertArg('ca', ca);
       c.context.addCACert(ca);
     }
   } else {
@@ -123,11 +123,11 @@ exports.createSecureContext = function createSecureContext(options) {
     if (Array.isArray(cert)) {
       for (i = 0; i < cert.length; ++i) {
         val = cert[i];
-        validateKeyCert('cert', val);
+        validateKeyCertArg('cert', val);
         c.context.setCert(val);
       }
     } else {
-      validateKeyCert('cert', cert);
+      validateKeyCertArg('cert', cert);
       c.context.setCert(cert);
     }
   }
@@ -144,11 +144,11 @@ exports.createSecureContext = function createSecureContext(options) {
         val = key[i];
         // eslint-disable-next-line eqeqeq
         const pem = (val != undefined && val.pem !== undefined ? val.pem : val);
-        validateKeyCert('key', pem);
+        validateKeyCertArg('key', pem);
         c.context.setKey(pem, val.passphrase || passphrase);
       }
     } else {
-      validateKeyCert('key', key);
+      validateKeyCertArg('key', key);
       c.context.setKey(key, passphrase);
     }
   }