http: don't cork .end when not needed

PR-URL: nodejs#36633 Backport-PR-URL: nodejs#36940 Fixes: nodejs#36620 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>
Pezmc · 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();
   }