fix(NA): MaxListenersExceededWarning on getLoggerStream (#57133) (#57257

) * fix(NA): possible EventEmitter memory leak detected with a passthrough for getLoggerStream * chore(na): remove passthrough Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
elastic · Feb 10, 2020 · 36b1dbb · 36b1dbb
1 parent 736c92c
commit 36b1dbb
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/legacy/server/logging/log_reporter.js b/src/legacy/server/logging/log_reporter.js
@@ -24,6 +24,14 @@ import LogFormatJson from './log_format_json';
 import LogFormatString from './log_format_string';
 import { LogInterceptor } from './log_interceptor';
 
+// NOTE: legacy logger creates a new stream for each new access
+// In https://github.com/elastic/kibana/pull/55937 we reach the max listeners
+// default limit of 10 for process.stdout which starts a long warning/error
+// thrown every time we start the server.
+// In order to keep using the legacy logger until we remove it I'm just adding
+// a new hard limit here.
+process.stdout.setMaxListeners(15);
+
 export function getLoggerStream({ events, config }) {
   const squeeze = new Squeeze(events);
   const format = config.json ? new LogFormatJson(config) : new LogFormatString(config);