fix: Fixed the issue that there is no running request when http2 goaw…

…ay (#3875)
nodejs · Nov 24, 2024 · 9b8abb8 · 9b8abb8
1 parent 43bfeb4
commit 9b8abb8
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/lib/dispatcher/client-h2.js b/lib/dispatcher/client-h2.js
@@ -203,11 +203,12 @@ function onHttp2SessionGoAway (errorCode) {
   util.destroy(this[kSocket], err)
 
   // Fail head of pipeline.
-  const request = client[kQueue][client[kRunningIdx]]
-  client[kQueue][client[kRunningIdx]++] = null
-  util.errorRequest(client, request, err)
-
-  client[kPendingIdx] = client[kRunningIdx]
+  if (client[kRunningIdx] < client[kQueue].length) {
+    const request = client[kQueue][client[kRunningIdx]]
+    client[kQueue][client[kRunningIdx]++] = null
+    util.errorRequest(client, request, err)
+    client[kPendingIdx] = client[kRunningIdx]
+  }
 
   assert(client[kRunning] === 0)