f29762f4dd
Enable linting for the test directory. A number of changes was made so all tests conform the current rules used by lib and src directories. The only exception for tests is that unreachable (dead) code is allowed. test-fs-non-number-arguments-throw had to be excluded from the changes because of a weird issue on Windows CI. PR-URL: https://github.com/nodejs/io.js/pull/1721 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
35 lines
983 B
JavaScript
35 lines
983 B
JavaScript
'use strict';
|
|
var common = require('../common');
|
|
|
|
if (!common.hasCrypto) {
|
|
console.log('1..0 # Skipped: missing crypto');
|
|
process.exit();
|
|
}
|
|
var tls = require('tls');
|
|
|
|
var assert = require('assert');
|
|
var fs = require('fs');
|
|
var path = require('path');
|
|
|
|
var cert = fs.readFileSync(path.join(common.fixturesDir, 'test_cert.pem'));
|
|
var key = fs.readFileSync(path.join(common.fixturesDir, 'test_key.pem'));
|
|
|
|
// https://github.com/nodejs/io.js/issues/1489
|
|
// tls.connect(options) with no options.host should accept a cert with
|
|
// CN:'localhost'
|
|
tls.createServer({
|
|
key: key,
|
|
cert: cert
|
|
}).listen(common.PORT);
|
|
|
|
var socket = tls.connect({
|
|
port: common.PORT,
|
|
ca: cert,
|
|
// No host set here. 'localhost' is the default,
|
|
// but tls.checkServerIdentity() breaks before the fix with:
|
|
// Error: Hostname/IP doesn't match certificate's altnames:
|
|
// "Host: undefined. is not cert's CN: localhost"
|
|
}, function() {
|
|
assert(socket.authorized);
|
|
process.exit();
|
|
});
|