fixup: reuse if not headersSent

nxtedition · Mar 8, 2020 · 6f11b24 · 6f11b24
1 parent 0245e4e
commit 6f11b24
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/lib/_http_client.js b/lib/_http_client.js
@@ -368,10 +368,7 @@ ClientRequest.prototype.destroy = function destroy(err) {
 };
 
 function _destroy(req, socket, err) {
-  // TODO (ronag): Check if socket was used at all (e.g. headersSent) and
-  // re-use it in that case. `req.socket` just checks whether the socket was
-  // assigned to the request and *might* have been used.
-  if (!req.agent || req.socket) {
+  if (!req.agent || req.headersSent) {
     socket.destroy(err);
   } else {
     socket.emit('free');

diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -335,6 +335,10 @@ function _writeRaw(data, encoding, callback) {
     return false;
   }
 
+  if (this.destroyed) {
+    return false;
+  }
+
   if (typeof encoding === 'function') {
     callback = encoding;
     encoding = null;