@lark-apaas/nestjs-logger 0.1.0-alpha.9 → 1.0.1-alpha.0

package/dist/index.js

@@ -10567,7 +10567,7 @@ var require_tap = __commonJS({
     var lift_1 = require_lift();
     var OperatorSubscriber_1 = require_OperatorSubscriber();
     var identity_1 = require_identity();
-    function tap2(observerOrNext, error, complete) {
+    function tap(observerOrNext, error, complete) {
       var tapObserver = isFunction_1.isFunction(observerOrNext) || error || complete ? {
         next: observerOrNext,
         error,
@@ -10600,8 +10600,8 @@ var require_tap = __commonJS({
         }));
       }) : identity_1.identity;
     }
-    __name(tap2, "tap");
-    exports.tap = tap2;
+    __name(tap, "tap");
+    exports.tap = tap;
   }
 });
 
@@ -14844,7 +14844,16 @@ var LoggingInterceptor = class {
       ip: req.ip ?? null,
       pid: process.pid
     };
-    this.appLogger.logStructured("log", "HTTP request started", baseMeta, "HTTPTraceInterceptor");
+    const originalJson = res.json.bind(res);
+    const originalSend = res.send.bind(res);
+    res.json = function(body) {
+      res.__finalResponseBody = body;
+      return originalJson(body);
+    };
+    res.send = function(body) {
+      res.__finalResponseBody = body;
+      return originalSend(body);
+    };
     const requestMeta = {
       ...baseMeta
     };
@@ -14854,50 +14863,36 @@ var LoggingInterceptor = class {
     if (this.config.logRequestBody && Object.keys(req.query || {}).length > 0) {
       requestMeta["query_params"] = this.sanitizeAndTruncate(req.query);
     }
+    this.appLogger.log("HTTP request started \n url=%s \n request_query=%o \n request_body=%o", req.url, requestMeta["query_params"] ?? {}, requestMeta["request_body"] ?? {});
     this.traceLogger.logStructured("verbose", "HTTP request started", requestMeta, "HTTPTraceInterceptor");
-    return next.handle().pipe((0, import_operators.tap)((responseData) => {
+    let logged = false;
+    res.on("finish", () => {
+      if (logged) {
+        return;
+      }
+      logged = true;
       const durationMs = Date.now() - startedAt;
       const statusCode = res.statusCode;
-      this.appLogger.logStructured("log", "HTTP request completed", {
-        ...baseMeta,
-        status_code: statusCode,
-        duration_ms: durationMs
-      }, "HTTPTraceInterceptor");
       const responseMeta = {
         ...baseMeta,
         status_code: statusCode,
         duration_ms: durationMs
       };
-      if (this.config.logResponseBody && responseData !== void 0) {
-        responseMeta["response_body"] = this.sanitizeAndTruncate(responseData);
+      const finalResponseBody = res.__finalResponseBody;
+      if (this.config.logResponseBody && finalResponseBody !== void 0) {
+        responseMeta["response_body"] = this.sanitizeAndTruncate(finalResponseBody);
       }
-      this.traceLogger.logStructured("verbose", "HTTP request completed", responseMeta, "HTTPTraceInterceptor");
-    }), (0, import_operators.catchError)((error) => {
-      const durationMs = Date.now() - startedAt;
-      const statusCode = res.statusCode >= 400 ? res.statusCode : 500;
-      const linkMeta = {
-        ...baseMeta,
-        status_code: statusCode,
-        duration_ms: durationMs
-      };
-      if (error instanceof Error) {
-        linkMeta["error_message"] = error.message;
-      }
-      this.appLogger.logStructured("log", "HTTP request failed", linkMeta, "HTTPTraceInterceptor");
-      const meta = {
-        ...baseMeta,
-        status_code: statusCode,
-        duration_ms: durationMs
-      };
-      if (error instanceof Error) {
-        meta["error"] = {
-          message: error.message,
-          stack: error.stack
-        };
+      const isError = statusCode >= 400;
+      const appLog = this.appLogger[isError ? "error" : "log"].bind(this.appLogger);
+      const traceLevel = isError ? "error" : "verbose";
+      if (isError) {
+        appLog("HTTP request failed\n url=%s \n status_code=%d \n duration_ms=%d \n response_body=%o", req.url, statusCode, durationMs, finalResponseBody ?? {});
       } else {
-        meta["error"] = String(error);
+        appLog("HTTP request completed\n url=%s \n status_code=%d \n duration_ms=%d \n response_body=%o", req.url, statusCode, durationMs, finalResponseBody ?? {});
       }
-      this.traceLogger.logStructured("verbose", "HTTP request failed", meta, "HTTPTraceInterceptor");
+      this.traceLogger.logStructured(traceLevel, isError ? "HTTP request failed" : "HTTP request completed", responseMeta, "HTTPTraceInterceptor");
+    });
+    return next.handle().pipe((0, import_operators.catchError)((error) => {
       throw error;
     }));
   }
@@ -14976,7 +14971,7 @@ function createPinoLogger(config) {
 }
 __name(createPinoLogger, "createPinoLogger");
 function createFileDestination(pathname) {
-  const target = pathname && pathname.length > 0 ? pathname : "logs/app.log";
+  const target = pathname && pathname.length > 0 ? pathname : "logs/server.log";
   const resolved = isAbsolute(target) ? target : join(process.cwd(), target);
   const dir = dirname(resolved);
   if (!existsSync(dir)) {
@@ -15065,7 +15060,7 @@ LoggerModule = _ts_decorate5([
         useFactory: /* @__PURE__ */ __name((config) => {
           return createPinoLogger({
             level: config.level,
-            filePath: `${config.logDir}/app.log`
+            filePath: `${config.logDir}/server.log`
           });
         }, "useFactory"),
         inject: [