fix: ignore headers timeout after statusCode

Refs: #268
nodejs · Jul 29, 2020 · 570bf41 · 570bf41
1 parent 0fbe4de
commit 570bf41
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/client.js b/lib/client.js
@@ -382,6 +382,10 @@ class Parser extends HTTPParser {
   }
 
   [HTTPParser.kOnTimeout] () {
+    if (this.statusCode) {
+      // https://github.com/mcollina/undici/issues/268
+      return
+    }
     util.destroy(this.socket, new HeadersTimeoutError())
   }
 
@@ -457,13 +461,13 @@ class Parser extends HTTPParser {
 
     assert(statusCode >= 100)
 
+    this.statusCode = null
+    this.headers = null
+
     if (statusCode < 200) {
       return
     }
 
-    this.statusCode = null
-    this.headers = null
-
     request.onComplete(headers)
 
     this.read = 0