fix printing logs as json (#7012)

teambit · Feb 7, 2023 · 3edfcaa · 3edfcaa
1 parent 350e2b1
commit 3edfcaa
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/src/logger/pino-logger.ts b/src/logger/pino-logger.ts
@@ -118,20 +118,29 @@ export function getPinoLoggerWithoutWorkers(
   });
 
   const prettyStream = prettifier({
-    ...(!jsonFormat ? prettyOptions : {}),
+    ...prettyOptions,
     destination: dest,
     sync: true,
   });
 
+  const fileStream = jsonFormat ? dest : prettyStream;
+
+  const destConsole = pino.destination({
+    sync: true, // no choice here :( otherwise, it looses data especially when an error is thrown (although pino.final is used to flush)
+  });
+
   const prettyConsoleStream = prettifier({
-    ...(!jsonFormat ? prettyOptionsConsole : {}),
+    ...prettyOptionsConsole,
     destination: 1,
     sync: true,
   });
 
-  const pinoLogger = pino(loggerOptions, prettyStream);
+  const consoleStream = jsonFormat ? destConsole : prettyConsoleStream;
+
+
+  const pinoLogger = pino(loggerOptions, fileStream);
 
-  const pinoLoggerConsole = pino(loggerOptions, prettyConsoleStream);
+  const pinoLoggerConsole = pino(loggerOptions, consoleStream);
 
   return { pinoLogger, pinoLoggerConsole };
 }