mirror of
https://github.com/nodejs/node.git
synced 2024-11-24 12:10:08 +01:00
38b82b0604
If we respond to a HEAD request with a body, we ignore all writes. However, we must still include all implicit headers. Fixes a regressions introduced in https://github.com/nodejs/node/pull/47732. Signed-off-by: Matteo Collina <hello@matteocollina.com> PR-URL: https://github.com/nodejs/node/pull/48108 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
28 lines
692 B
JavaScript
28 lines
692 B
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
const http = require('http');
|
|
|
|
// This test is to make sure that when the HTTP server
|
|
// responds to a HEAD request with data to res.end,
|
|
// it does not send any body but the response is sent
|
|
// anyway.
|
|
|
|
const server = http.createServer(function(req, res) {
|
|
res.end('FAIL'); // broken: sends FAIL from hot path.
|
|
});
|
|
server.listen(0);
|
|
|
|
server.on('listening', common.mustCall(function() {
|
|
const req = http.request({
|
|
port: this.address().port,
|
|
method: 'HEAD',
|
|
path: '/'
|
|
}, common.mustCall(function(res) {
|
|
res.on('end', common.mustCall(function() {
|
|
server.close();
|
|
}));
|
|
res.resume();
|
|
}));
|
|
req.end();
|
|
}));
|