mirror of
https://github.com/nodejs/node.git
synced 2025-05-06 09:02:40 +00:00
5f76b24e5e
Make default `clientError` behavior (close socket immediately)
overridable. With this APIs it is possible to write a custom error
handler, and to send, for example, a 400 HTTP response.
http.createServer(...).on('clientError', function(err, socket) {
socket.end('HTTP/1.1 400 Bad Request\r\n\r\n');
socket.destroy();
});
Fix: #4543
PR-URL: https://github.com/nodejs/node/pull/4557
Reviewed-By: Brian White <mscdex@mscdex.net>
41 lines
1.0 KiB
JavaScript
41 lines
1.0 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 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++;
|
|
conn.destroy();
|
|
});
|
|
|
|
server.listen(common.PORT, function() {
|
|
net.connect({ host: '127.0.0.1', port: common.PORT });
|
|
});
|