Revert "http: do not leak error listeners"

This reverts commit 736a7d8. The patch attempted to fix an issue, signaled by a warning, caused by an invalid API usage. However it introduced a behavior change that is breaking userland modules. It is a user error, so restore the original behavior and have the user investigate and fix the issue in their code. Refs: nodejs#43587 (comment) Refs: nodejs#43587 (comment)
lpinca · Oct 8, 2022 · 03232eb · 03232eb
1 parent 7e09c6c
commit 03232eb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 48 deletions.
diff --git a/lib/_http_server.js b/lib/_http_server.js
@@ -809,10 +809,7 @@ const requestHeaderFieldsTooLargeResponse = Buffer.from(
 function socketOnError(e) {
   // Ignore further errors
   this.removeListener('error', socketOnError);
-
-  if (this.listenerCount('error') === 0) {
-    this.on('error', noop);
-  }
+  this.on('error', noop);
 
   if (!this.server.emit('clientError', e, this)) {
     // Caution must be taken to avoid corrupting the remote peer.

diff --git a/test/parallel/test-http-socket-listeners.js b/test/parallel/test-http-socket-listeners.js