http: don't cork .end when not needed

PR-URL: #36633 Backport-PR-URL: #36940 Fixes: #36620 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Danielle Adams <[email protected]>
nodejs · Jan 26, 2021 · 5593187 · 5593187
1 parent f002f23
commit 5593187
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -775,7 +775,8 @@ OutgoingMessage.prototype.end = function end(chunk, encoding, callback) {
     encoding = null;
   }
 
-  if (this.socket) {
+  // Not finished, socket exists and data will be written (chunk or header)
+  if (this.socket && !this.finished && (chunk || !this._header)) {
     this.socket.cork();
   }