Merge pull request #3350 from ulrichb/SingleWriteInLogger

fix(common): single `stdout.write()` call per logging in default `Logger`

packages/common/services/logger.service.ts

@@ -142,29 +142,31 @@ export class Logger implements LoggerService {
       undefined,
       localeStringOptions,
     );
-    process.stdout.write(color(`[Nest] ${process.pid}   - `));
-    process.stdout.write(`${timestamp}   `);
 
-    context && process.stdout.write(yellow(`[${context}] `));
-    process.stdout.write(output);
+    const pidMessage = color(`[Nest] ${process.pid}   - `);
+    const contextMessage = context ? yellow(`[${context}] `) : '';
+    const timestampDiff = this.updateAndGetTimestampDiff(isTimeDiffEnabled);
 
-    this.printTimestamp(isTimeDiffEnabled);
-    process.stdout.write(`\n`);
+    process.stdout.write(
+      `${pidMessage}${timestamp}   ${contextMessage}${output}${timestampDiff}\n`,
+    );
   }
 
-  private static printTimestamp(isTimeDiffEnabled?: boolean) {
+  private static updateAndGetTimestampDiff(
+    isTimeDiffEnabled?: boolean,
+  ): string {
     const includeTimestamp = Logger.lastTimestamp && isTimeDiffEnabled;
-    if (includeTimestamp) {
-      process.stdout.write(yellow(` +${Date.now() - Logger.lastTimestamp}ms`));
-    }
+    const result = includeTimestamp
+      ? yellow(` +${Date.now() - Logger.lastTimestamp}ms`)
+      : '';
     Logger.lastTimestamp = Date.now();
+    return result;
   }
 
   private static printStackTrace(trace: string) {
     if (!trace) {
       return;
     }
-    process.stdout.write(trace);
-    process.stdout.write(`\n`);
+    process.stdout.write(`${trace}\n`);
   }
 }