fix(common): Fix error on requestContext that disable logs

Closes: #764

Author: Romakita

packages/common/src/mvc/models/RequestContext.ts

@@ -83,6 +83,6 @@ export class RequestContext extends Map<any, any> {
   }
 
   async emit(eventName: string, ...args: any[]) {
-    return this.injector && this.injector.emit(eventName, args);
+    return this.injector && this.injector.emit(eventName, ...args);
   }
 }

packages/common/src/server/components/LogIncomingRequestMiddleware.ts

@@ -72,6 +72,7 @@ export class LogIncomingRequestMiddleware implements IMiddleware {
         });
       }
     }
+
     request.ctx.logger.flush();
   }
 

packages/common/src/server/utils/contextMiddleware.ts

@@ -21,6 +21,16 @@ export class ContextMiddleware {
     this.reqIdBuilder = reqIdBuilder;
   }
 
+  static async onClose(err: any, response: any) {
+    const {req: request} = response;
+
+    await request.ctx.emit("$onResponse", request, response);
+    await request.ctx.destroy();
+
+    delete request.ctx;
+    delete request.log;
+  }
+
   async use(request: any, response: any, next: any) {
     const {level, ignoreUrlPatterns, maxStackSize} = this;
 
@@ -45,17 +55,6 @@ export class ContextMiddleware {
 
     next();
   }
-
-  static async onClose(err: any, response: any) {
-    const {req: request} = response;
-    try {
-      await request.ctx.emit("$onResponse", request, response);
-      await request.ctx.destroy();
-    } catch (er) {}
-
-    delete request.ctx;
-    delete request.log;
-  }
 }
 
 /**

tasks/gulp/clean.js

@@ -19,7 +19,7 @@ module.exports = {
       .src([
         "test/**/*.{js,js.map,d.ts}",
         "test/**/*.{js,js.map,d.ts}",
-        "packages/**/*.{js,js.map,d.ts,d.ts.map}",
+        "packages/**/src/**/*.{js,js.map,d.ts,d.ts.map}",
         "packages/**/node_modules"
       ], {read: false})
       .pipe(clean());