60de9f8d7b
Currently there are many instances where assert.fail is directly passed to a callback for error handling. Unfortunately this will swallow the error as it is the third argument of assert.fail that sets the message not the first. This commit adds a new function to test/common.js that simply wraps assert.fail and calls it with the provided message. Tip of the hat to @trott for pointing me in the direction of this. PR-URL: https://github.com/nodejs/node/pull/3453 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
40 lines
1 KiB
JavaScript
40 lines
1 KiB
JavaScript
'use strict';
|
|
var common = require('../common');
|
|
var assert = require('assert');
|
|
|
|
if (!common.hasCrypto) {
|
|
console.log('1..0 # Skipped: missing crypto');
|
|
return;
|
|
}
|
|
var https = require('https');
|
|
|
|
var net = require('net');
|
|
var tls = require('tls');
|
|
var fs = require('fs');
|
|
|
|
var clientErrors = 0;
|
|
|
|
process.on('exit', function() {
|
|
assert.equal(clientErrors, 1);
|
|
});
|
|
|
|
var options = {
|
|
key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'),
|
|
cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem'),
|
|
handshakeTimeout: 50
|
|
};
|
|
|
|
var server = https.createServer(options, common.fail);
|
|
|
|
server.on('clientError', function(err, conn) {
|
|
// Don't hesitate to update the asserts if the internal structure of
|
|
// the cleartext object ever changes. We're checking that the https.Server
|
|
// has closed the client connection.
|
|
assert.equal(conn._secureEstablished, false);
|
|
server.close();
|
|
clientErrors++;
|
|
});
|
|
|
|
server.listen(common.PORT, function() {
|
|
net.connect({ host: '127.0.0.1', port: common.PORT });
|
|
});
|