From b3172f834f418f4f2656d851e585f17aece73333 Mon Sep 17 00:00:00 2001 From: Matteo Collina Date: Thu, 22 Aug 2019 11:42:46 +0200 Subject: [PATCH] test, http: add regression test for keepalive 'end' event This test covers a regression where 'end' was not emitted in the case of keepalive requests without parsing the full body. PR-URL: https://github.com/nodejs/node/pull/29263 Reviewed-By: Benjamin Gruenbaum Reviewed-By: James M Snell Reviewed-By: Luigi Pinca --- .../test-http-server-keepalive-end.js | 29 +++++++++++++++++++ 1 file changed, 29 insertions(+) create mode 100644 test/parallel/test-http-server-keepalive-end.js diff --git a/test/parallel/test-http-server-keepalive-end.js b/test/parallel/test-http-server-keepalive-end.js new file mode 100644 index 00000000000000..1e3a44d368a020 --- /dev/null +++ b/test/parallel/test-http-server-keepalive-end.js @@ -0,0 +1,29 @@ +'use strict'; + +const common = require('../common'); +const { createServer } = require('http'); +const { connect } = require('net'); + +const server = createServer(common.mustCall((req, res) => { + req.on('end', common.mustCall()); + res.end('hello world'); +})); + +server.unref(); + +server.listen(0, common.mustCall(() => { + + const client = connect(server.address().port); + + const req = [ + 'POST / HTTP/1.1', + `Host: localhost:${server.address().port}`, + 'Connection: keep-alive', + 'Content-Length: 11', + '', + 'hello world', + '' + ].join('\r\n'); + + client.end(req); +}));