fix(fetch): remove terminated event listener on redirect (#1715)

* fix(fetch): remove terminated event listener on redirect * fix: use setMaxListeners instead * Update lib/fetch/index.js Co-authored-by: Robert Nagy <ronagy@icloud.com>
nodejs · Oct 26, 2022 · 9567c62 · 9567c62
1 parent 3d7de94
commit 9567c62
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/lib/fetch/index.js b/lib/fetch/index.js
@@ -74,6 +74,12 @@ class Fetch extends EE {
     this.connection = null
     this.dump = false
     this.state = 'ongoing'
+    // 2 terminated listeners get added per request,
+    // but only 1 gets removed. If there are 20 redirects,
+    // 21 listeners will be added.
+    // See https://github.com/nodejs/undici/issues/1711
+    // TODO (fix): Find and fix root cause for leaked listener.
+    this.setMaxListeners(21)
   }
 
   terminate (reason) {