doc: fix inconsistent documentation (host vs hostname) · nodejs/node@9f9355d

return true;

}

exports.checkServerIdentity = function checkServerIdentity(host, cert) {

exports.checkServerIdentity = function checkServerIdentity(hostname, cert) {

const subject = cert.subject;

const altNames = cert.subjectaltname;

const dnsNames = [];

const uriNames = [];

const ips = [];

host = '' + host;

hostname = '' + hostname;

if (altNames) {

for (const name of altNames.split(', ')) {

let valid = false;

let reason = 'Unknown reason';

if (net.isIP(host)) {

valid = ips.includes(canonicalizeIP(host));

if (net.isIP(hostname)) {

valid = ips.includes(canonicalizeIP(hostname));

if (!valid)

reason = `IP: ${host} is not in the cert's list: ${ips.join(', ')}`;

reason = `IP: ${hostname} is not in the cert's list: ${ips.join(', ')}`;

// TODO(bnoordhuis) Also check URI SANs that are IP addresses.

} else if (subject) {

host = unfqdn(host); // Remove trailing dot for error messages.

const hostParts = splitHost(host);

hostname = unfqdn(hostname); // Remove trailing dot for error messages.

const hostParts = splitHost(hostname);

const wildcard = (pattern) => check(hostParts, pattern, true);

const noWildcard = (pattern) => check(hostParts, pattern, false);

valid = wildcard(cn);

if (!valid)

reason = `Host: ${host}. is not cert's CN: ${cn}`;

reason = `Host: ${hostname}. is not cert's CN: ${cn}`;

} else {

valid = dnsNames.some(wildcard) || uriNames.some(noWildcard);

if (!valid)

reason = `Host: ${host}. is not in the cert's altnames: ${altNames}`;

reason =

`Host: ${hostname}. is not in the cert's altnames: ${altNames}`;

}

} else {

reason = 'Cert is empty';

if (!valid) {

const err = new ERR_TLS_CERT_ALTNAME_INVALID(reason);

err.reason = reason;

err.host = host;

err.host = hostname;

err.cert = cert;

return err;

}