fix: avoid mem leak

Refs: nodejs/node#46435
nodejs · Mar 26, 2023 · a3f6e87 · a3f6e87
1 parent f0271d4
commit a3f6e87
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/fetch/request.js b/lib/fetch/request.js
@@ -354,8 +354,12 @@ class Request {
       if (signal.aborted) {
         ac.abort(signal.reason)
       } else {
+        const weakSelf = new WeakRef(this)
         const abort = function () {
-          ac.abort(this.reason)
+          const self = weakSelf.deref()
+          if (self) {
+            ac.abort(self.reason)
+          }
         }
 
         // Third-party AbortControllers may not work with these.