Merge pull request #1089 from jhonatanTeixeira/fix_tcp_error_newer_node

fix: on newer nodejs versions listening to socket errors are required
log4js-node · Jan 5, 2022 · 09b43b1 · 09b43b1
2 parents 0144f9a + 3f1ac52
commit 09b43b1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/appenders/tcp.js b/lib/appenders/tcp.js
@@ -39,7 +39,11 @@ function appender(config, layout) {
       emptyBuffer();
     });
     socket.on('timeout', socket.end.bind(socket));
-    // don't bother listening for 'error', 'close' gets called after that anyway
+    socket.on('error', (e) => {
+      debug('connection error', e);
+      canWrite = false;
+      emptyBuffer();
+    });
     socket.on('close', createSocket);
   }