mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 21:19:50 +01:00
ed3604cd64
PR-URL: https://github.com/nodejs/node/pull/45597 Fixes: https://github.com/nodejs/node/issues/39033 Co-authored-by: Luigi Pinca <luigipinca@gmail.com> Co-authored-by: mscdex <mscdex@users.noreply.github.com> Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
100 lines
2.5 KiB
JavaScript
100 lines
2.5 KiB
JavaScript
'use strict';
|
|
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
const { createServer } = require('http');
|
|
const { connect } = require('net');
|
|
|
|
// This test validates that the server returns 408
|
|
// after server.headersTimeout if the client
|
|
// does not send headers before the timeout, and
|
|
// that keep alive works properly.
|
|
|
|
function performRequestWithDelay(client, firstDelay, secondDelay, closeAfter) {
|
|
client.resume();
|
|
client.write('GET / HTTP/1.1\r\nHost: example.com\r\n');
|
|
|
|
setTimeout(() => {
|
|
client.write('Connection: ');
|
|
}, firstDelay).unref();
|
|
|
|
// Complete the request
|
|
setTimeout(() => {
|
|
client.write(`${closeAfter ? 'close' : 'keep-alive'}\r\n\r\n`);
|
|
}, firstDelay + secondDelay).unref();
|
|
}
|
|
|
|
const headersTimeout = common.platformTimeout(5000);
|
|
const server = createServer({
|
|
headersTimeout,
|
|
requestTimeout: 0,
|
|
keepAliveTimeout: 0,
|
|
connectionsCheckingInterval: headersTimeout / 4,
|
|
}, common.mustCallAtLeast((req, res) => {
|
|
res.writeHead(200, { 'Content-Type': 'text/plain' });
|
|
res.end();
|
|
}));
|
|
|
|
assert.strictEqual(server.headersTimeout, headersTimeout);
|
|
|
|
// Check that timeout event is not triggered
|
|
server.once('timeout', common.mustNotCall((socket) => {
|
|
socket.destroy();
|
|
}));
|
|
|
|
server.listen(0, common.mustCall(() => {
|
|
const client = connect(server.address().port);
|
|
let second = false;
|
|
let response = '';
|
|
|
|
client.setEncoding('utf8');
|
|
client.on('data', common.mustCallAtLeast((chunk) => {
|
|
response += chunk;
|
|
|
|
// First response has ended
|
|
if (!second && response.endsWith('\r\n\r\n')) {
|
|
assert.strictEqual(
|
|
response.split('\r\n')[0],
|
|
'HTTP/1.1 200 OK'
|
|
);
|
|
|
|
const defer = headersTimeout * 1.5;
|
|
|
|
// Wait some time to make sure headersTimeout
|
|
// does not interfere with keep alive
|
|
setTimeout(() => {
|
|
response = '';
|
|
second = true;
|
|
|
|
// Perform a second request expected to finish after headersTimeout
|
|
performRequestWithDelay(
|
|
client,
|
|
headersTimeout / 5,
|
|
headersTimeout,
|
|
true
|
|
);
|
|
}, defer).unref();
|
|
}
|
|
}, 1));
|
|
|
|
const errOrEnd = common.mustCall(function(err) {
|
|
assert.strictEqual(second, true);
|
|
assert.strictEqual(
|
|
response,
|
|
'HTTP/1.1 408 Request Timeout\r\nConnection: close\r\n\r\n'
|
|
);
|
|
server.close();
|
|
});
|
|
|
|
client.on('error', errOrEnd);
|
|
client.on('end', errOrEnd);
|
|
|
|
// Perform a second request expected to finish before headersTimeout
|
|
performRequestWithDelay(
|
|
client,
|
|
headersTimeout / 5,
|
|
headersTimeout / 5,
|
|
false
|
|
);
|
|
}));
|