Workaround for GitHub issue request#2807 (request#2808)

shotamakino · Dec 1, 2018 · 2f04c3c · 2f04c3c
1 parent 43c53ce
commit 2f04c3c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 167 deletions.
diff --git a/lib/auth.js b/lib/auth.js
diff --git a/request.js b/request.js
@@ -878,6 +878,16 @@ Request.prototype.onRequestError = function (error) {
     clearTimeout(self.timeoutTimer)
     self.timeoutTimer = null
   }
+
+  // This is a workaround for a race condition caused by the way that lib/redirect.js
+  // calls Request.init() when processing an HTTP redirect:
+  // https://github.com/request/request/issues/2807
+  // Somehow we end up with an ECONNRESET error from a socket that has already
+  // been destroyed and returned to the pool.
+  if (self.req && self.req.socket && self.req.socket.destroyed) {
+    return
+  }
+
   self.emit('error', error)
 }