René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"HTTP parser premature close","articleBody":"\u003c!--\r\nThank you for reporting a possible bug in Node.js.\r\n\r\nPlease fill in as much of the template below as you can.\r\n\r\nVersion: output of `node -v`\r\nPlatform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)\r\nSubsystem: if known, please specify the affected core module name\r\n\r\nIf possible, please provide code that demonstrates the problem, keeping it as\r\nsimple and free of external dependencies as you can.\r\n--\u003e\r\n\r\n* **Version**: v12.10.0\r\n* **Platform**: macOS\r\n* **Subsystem**: http, stream\r\n\r\n\u003c!-- Please provide more details below this comment. --\u003e\r\n\r\n\r\nI'm running across an issue where when no response data is written during the \"request\" event call, the HTTP handling either closes the response without sending any apparent reply, or sends a `Connection: close` response but doesn't end the stream.\r\n\r\nThis is a strange behavior that I can't find an explanation for and I can't come up with any workaround; so I think this is a bug.\r\n\r\n**Case 1**\r\n\r\n```javascript\r\n\"use strict\";\r\nconst http = require('http');\r\nconst makeDuplexPair = require('./test/common/duplexpair.js');\r\n\r\nconst message = 'GET / HTTP/1.1\\r\\nHost: example.com\\r\\nConnection: close\\r\\n\\r\\n';\r\nconst server = http.createServer(function(req, res){\r\n\tconsole.log('Have request: ', req.method, req.url);\r\n\treq.resume();\r\n\treq.once('end', function(){\r\n\t\tconsole.log('(request end)');\r\n\t\tres.end('Fin\\r\\n');\r\n\t});\r\n});\r\n\r\nconst {clientSide, serverSide} = makeDuplexPair();\r\nclientSide.end(message);\r\nclientSide.pipe(process.stdout);\r\nclientSide.once('end', function(){ console.log('(Connection closed)'); });\r\n\r\nserver.emit('connection', serverSide);\r\n```\r\n\r\nActual:\r\n```\r\nHave request:  GET /\r\n(request end)\r\n(Connection closed)\r\n```\r\n\r\nI expect to see the HTTP response output, but the connection simply closes.\r\n\r\n\r\n**Case 2** (using clientSide.write instead of clientSide.end)\r\n```javascript\r\n\"use strict\";\r\nconst http = require('http');\r\nconst makeDuplexPair = require('./test/common/duplexpair.js');\r\n\r\nconst message = 'GET / HTTP/1.1\\r\\nHost: example.com\\r\\nConnection: close\\r\\n\\r\\n';\r\nconst server = http.createServer(function(req, res){\r\n\tconsole.log('Have request: ', req.method, req.url);\r\n\treq.resume();\r\n\treq.once('end', function(){\r\n\t\tconsole.log('(request end)');\r\n\t\tres.end('Fin\\r\\n');\r\n\t});\r\n});\r\n\r\nconst {clientSide, serverSide} = makeDuplexPair();\r\nclientSide.write(message);\r\nclientSide.pipe(process.stdout);\r\nclientSide.once('end', function(){ console.log('(Connection closed)'); });\r\n\r\nserver.emit('connection', serverSide);\r\n```\r\n\r\nI expect to see `(Connection closed)` at the end, but I do not.\r\n\r\nActual:\r\n```\r\nHave request:  GET /\r\n(request end)\r\nHTTP/1.1 200 OK\r\nDate: Thu, 19 Sep 2019 01:01:49 GMT\r\nConnection: close\r\nContent-Length: 5\r\n\r\nFin\r\n```\r\n\r\n\r\nIn one or the other case I expect:\r\n```\r\nHave request:  GET /\r\n(request end)\r\nHTTP/1.1 200 OK\r\nDate: Thu, 19 Sep 2019 01:01:49 GMT\r\nConnection: close\r\nContent-Length: 5\r\n\r\nFin\r\n(Connection closed)\r\n```\r\n\r\nAs far as I can tell, there's no way to do this.","author":{"url":"https://github.com/awwright","@type":"Person","name":"awwright"},"datePublished":"2019-09-19T01:04:41.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":8},"url":"https://github.com/29609/node/issues/29609"}