mirror of
https://github.com/nodejs/node.git
synced 2025-04-29 06:19:07 +00:00

Lots of changes, but mostly just search/replace of fixtures.readSync(...) to fixtures.readKey([new key]...) Benchmarks modified to use fixtures.readKey(...): benchmark/tls/throughput.js benchmark/tls/tls-connect.js benchmark/tls/secure-pair.js Also be sure to review the change to L16 of test/parallel/test-crypto-sign-verify.js PR-URL: https://github.com/nodejs/node/pull/27962 Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
34 lines
977 B
JavaScript
34 lines
977 B
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
const fixtures = require('../common/fixtures');
|
|
|
|
if (!common.hasCrypto)
|
|
common.skip('missing crypto');
|
|
|
|
const tls = require('tls');
|
|
const assert = require('assert');
|
|
|
|
const cert = fixtures.readKey('rsa_cert.crt');
|
|
const key = fixtures.readKey('rsa_private.pem');
|
|
|
|
// https://github.com/nodejs/node/issues/1489
|
|
// tls.connect(options) with no options.host should accept a cert with
|
|
// CN:'localhost'
|
|
const server = tls.createServer({
|
|
key,
|
|
cert
|
|
}).listen(0, common.mustCall(function() {
|
|
const socket = tls.connect({
|
|
port: this.address().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"
|
|
}, common.mustCall(function() {
|
|
assert(socket.authorized);
|
|
socket.destroy();
|
|
server.close();
|
|
}));
|
|
}));
|