fix(di): fix issue when pushing logs after request end (#2423)

tsedio · Sep 2, 2023 · 7e26465 · 7e26465
1 parent 77a8649
commit 7e26465
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/packages/di/jest.config.js b/packages/di/jest.config.js
@@ -6,9 +6,9 @@ module.exports = {
   coverageThreshold: {
     global: {
       statements: 98.81,
-      branches: 96.54,
+      branches: 96.55,
       lines: 98.81,
-      functions: 98.32
+      functions: 98.34
     }
   }
 };
diff --git a/packages/di/src/node/domain/ContextLogger.ts b/packages/di/src/node/domain/ContextLogger.ts
@@ -79,14 +79,18 @@ export class ContextLogger {
     return this;
   }
 
-  public flush() {
+  public flush(stream = false) {
     if (this.stack.length) {
       this.stack.forEach(({level, data}: any) => {
         this.#logger[level](data);
       });
 
       this.#stack = [];
     }
+
+    if (stream) {
+      this.maxStackSize = 0;
+    }
   }
 
   public isLevelEnabled(otherLevel: string | LogLevel) {

diff --git a/packages/di/src/node/domain/DIContext.ts b/packages/di/src/node/domain/DIContext.ts
@@ -73,7 +73,7 @@ export class DIContext {
   }
 
   destroy(): Promise<any> {
-    return Promise.all([this.#container?.destroy(), this.#logger?.flush()]);
+    return Promise.all([this.#container?.destroy(), this.#logger?.flush(true)]);
   }
 
   emit(eventName: string, ...args: any[]) {

diff --git a/packages/platform/platform-log-middleware/src/middlewares/PlatformLogMiddleware.ts b/packages/platform/platform-log-middleware/src/middlewares/PlatformLogMiddleware.ts
@@ -81,8 +81,6 @@ export class PlatformLogMiddleware implements MiddlewareMethods {
         }
       }
     }
-
-    ctx.logger.flush();
   }
 
   /**