refactor: simplify signal handling (#3362)

Author: ronag

lib/api/api-request.js

@@ -77,12 +77,6 @@ class RequestHandler extends AsyncResource {
           } else if (this.abort) {
             this.abort(this.reason)
           }
-
-          if (this.removeAbortListener) {
-            this.res?.off('close', this.removeAbortListener)
-            this.removeAbortListener()
-            this.removeAbortListener = null
-          }
         })
       }
     }
@@ -127,6 +121,7 @@ class RequestHandler extends AsyncResource {
 
     if (this.removeAbortListener) {
       res.on('close', this.removeAbortListener)
+      this.removeAbortListener = null
     }
 
     this.callback = null
@@ -183,7 +178,6 @@ class RequestHandler extends AsyncResource {
     }
 
     if (this.removeAbortListener) {
-      res?.off('close', this.removeAbortListener)
       this.removeAbortListener()
       this.removeAbortListener = null
     }

lib/core/util.js

@@ -518,7 +518,7 @@ function addAbortListener (signal, listener) {
     signal.addEventListener('abort', listener, { once: true })
     return () => signal.removeEventListener('abort', listener)
   }
-  signal.addListener('abort', listener)
+  signal.once('abort', listener)
   return () => signal.removeListener('abort', listener)
 }