sendRequest: clean up error/response event handlers upon failure/succ…

…ess 🐛

lib/send-request.js

@@ -153,7 +153,7 @@ const createSendRequest = (cfg, opt = {}) => {
 		logCtx.clientRequest = req
 
 		const res = await new Promise((resolve, reject) => {
-			req.once('error', (err) => {
+			const onError = (err) => {
 				reject(err)
 				clearTimeout(abortTimer)
 
@@ -162,8 +162,16 @@ const createSendRequest = (cfg, opt = {}) => {
 					err,
 				}, `failed to send client request: ${err.message}`)
 				req.destroy(err)
-			})
-			req.once('response', resolve)
+			}
+
+			// todo: is this necessary for req/res to be GCed?
+			const cleanUpAndResolve = (response) => {
+				req.removeListener('error', onError)
+				req.removeListener('response', cleanUpAndResolve)
+				resolve(response)
+			}
+			req.on('error', onError)
+			req.on('response', cleanUpAndResolve)
 
 			req.end(reqBody) // send request body
 		})