1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

tls: allow wildcards in common name

Browse Source 
see #4592
This commit is contained in:
Fedor Indutny 2013-01-14 21:09:35 +04:00
parent f3e78bd3c2
commit 45024e7b75
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/tls.js
View File

@ -156,14 +156,13 @@ function checkServerIdentity(host, cert) {
dnsNames = dnsNames.concat(uriNames); dnsNames = dnsNames.concat(uriNames);
// And only after check if hostname matches CN // And only after check if hostname matches CN
// (because CN is deprecated, but should be used for compatiblity anyway)
var commonNames = cert.subject.CN; var commonNames = cert.subject.CN;
if (Array.isArray(commonNames)) { if (Array.isArray(commonNames)) {
for (var i = 0, k = commonNames.length; i < k; ++i) { for (var i = 0, k = commonNames.length; i < k; ++i) {
dnsNames.push(regexpify(commonNames[i], false)); dnsNames.push(regexpify(commonNames[i], true));
} }
} else { } else {
dnsNames.push(regexpify(commonNames, false)); dnsNames.push(regexpify(commonNames, true));
} }
valid = dnsNames.some(function(re) { valid = dnsNames.some(function(re) {