@lark-apaas/nestjs-logger 1.0.4 → 1.0.5

package/dist/index.d.cts

@@ -25,6 +25,7 @@ declare abstract class BasePinoLogger implements LoggerService {
      */
     logStructured(level: LogLevel, message: string, meta: Record<string, unknown>, context?: string): void;
     protected write(level: LogLevel, message: unknown, optionalParams: unknown[], treatStack?: boolean): void;
+    private processLogParams;
     private extractOptionalParams;
     private buildMessagePayload;
     private static looksLikeStack;

package/dist/index.d.ts

@@ -25,6 +25,7 @@ declare abstract class BasePinoLogger implements LoggerService {
      */
     logStructured(level: LogLevel, message: string, meta: Record<string, unknown>, context?: string): void;
     protected write(level: LogLevel, message: unknown, optionalParams: unknown[], treatStack?: boolean): void;
+    private processLogParams;
     private extractOptionalParams;
     private buildMessagePayload;
     private static looksLikeStack;

package/dist/index.js

@@ -7241,7 +7241,7 @@ var require_tap = __commonJS({
     var lift_1 = require_lift();
     var OperatorSubscriber_1 = require_OperatorSubscriber();
     var identity_1 = require_identity();
-    function tap(observerOrNext, error, complete) {
+    function tap2(observerOrNext, error, complete) {
       var tapObserver = isFunction_1.isFunction(observerOrNext) || error || complete ? {
         next: observerOrNext,
         error,
@@ -7274,8 +7274,8 @@ var require_tap = __commonJS({
         }));
       }) : identity_1.identity;
     }
-    __name(tap, "tap");
-    exports.tap = tap;
+    __name(tap2, "tap");
+    exports.tap = tap2;
   }
 });
 
@@ -9000,6 +9000,47 @@ var OBSERVABLE_LOGGER_SERVICE = /* @__PURE__ */ Symbol("OBSERVABLE_LOGGER_SERVIC
 
 // src/service/app-logger.service.ts
 import { RequestContextService, ObservableService } from "@lark-apaas/nestjs-common";
+
+// src/utils/safeStringify.ts
+function safeStringify(obj) {
+  const seen = /* @__PURE__ */ new Set();
+  try {
+    return JSON.stringify(obj, (key, value) => {
+      if (typeof value === "object" && value !== null) {
+        if (seen.has(value)) {
+          return "[Circular]";
+        }
+        seen.add(value);
+      }
+      if (typeof value === "bigint") {
+        return value.toString();
+      }
+      if (value instanceof Date) {
+        return value.toISOString();
+      }
+      if (value instanceof Map) {
+        return Object.fromEntries(value);
+      }
+      if (value instanceof Set) {
+        return Array.from(value);
+      }
+      if (typeof value === "undefined") {
+        return "undefined";
+      }
+      if (typeof value === "symbol") {
+        return value.toString();
+      }
+      return value;
+    });
+  } catch {
+    return "";
+  } finally {
+    seen.clear();
+  }
+}
+__name(safeStringify, "safeStringify");
+
+// src/service/app-logger.service.ts
 function _ts_decorate(decorators, target, key, desc) {
   var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
   if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
@@ -9183,6 +9224,7 @@ var BasePinoLogger = class _BasePinoLogger {
     const { context, stack, extras } = this.extractOptionalParams(optionalParams, treatStack);
     const requestState = this.contextStore.getContext();
     const traceId = requestState?.requestId ?? null;
+    const firstParam = optionalParams[0];
     const flatObject = /* @__PURE__ */ __name((extra) => {
       let res = {};
       try {
@@ -9227,16 +9269,50 @@ var BasePinoLogger = class _BasePinoLogger {
     }
     const pinoLevel = mapLogLevelToPino(level);
     const sanitizedPayload = sanitizeValue(payload);
+    const isTriggerLog = firstParam?.module === "trigger";
+    const platformModule = isTriggerLog ? "trigger" : firstParam?.paas_attributes_module;
     if (process.env.NODE_ENV === "development") {
+      if (platformModule) {
+        return;
+      }
       if (messageText) {
         this.logger[pinoLevel](sanitizedPayload, messageText);
       } else {
         this.logger[pinoLevel](sanitizedPayload);
       }
     } else {
-      this.observableService.log(level, messageText ?? "", payload);
+      let finalMessage = messageText;
+      if (platformModule) {
+        if (typeof message !== "string") {
+          finalMessage = safeStringify(message);
+        }
+        this.observableService.log(level, finalMessage ?? "", {
+          ...payload,
+          module: platformModule
+        });
+      } else {
+        const reportMessage = this.processLogParams([
+          message,
+          ...optionalParams
+        ]);
+        this.observableService.log(level, reportMessage, {});
+      }
     }
   }
+  processLogParams(args) {
+    if (args.length === 1) {
+      const firstParam = args[0];
+      if (typeof firstParam !== "string") {
+        return safeStringify(firstParam);
+      }
+      return firstParam;
+    }
+    const obj = {};
+    for (let i = 0; i < args.length; i++) {
+      obj[i.toString()] = args[i];
+    }
+    return safeStringify(obj);
+  }
   extractOptionalParams(optionalParams, treatStack) {
     const params = [
       ...optionalParams
@@ -9434,9 +9510,14 @@ var LoggingInterceptor = class {
     if (context.getType() !== "http") {
       return next.handle();
     }
+    const isDev = process.env.NODE_ENV === "development";
     const httpContext = context.switchToHttp();
     const req = httpContext.getRequest();
     const res = httpContext.getResponse();
+    const protocol = req.protocol;
+    const host = req.get("Host");
+    const originalUrl = req.originalUrl;
+    const fullUrl = `${protocol}://${host}${originalUrl}`;
     const method = req.method;
     const path = req.path;
     this.requestContext.setContext({
@@ -9473,8 +9554,10 @@ 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
+    if (isDev) {
+      this.appLogger.log(`HTTP request started
 url=${req.url}`, "HTTPTraceInterceptor");
+    }
     this.traceLogger.logStructured("verbose", "HTTP request started", requestMeta, "HTTPTraceInterceptor");
     let logged = false;
     res.on("finish", () => {
@@ -9512,7 +9595,7 @@ url=${req.url}
 status_code=${statusCode}
 duration_ms=${durationMs}`, "HTTPTraceInterceptor");
         }
-      } else {
+      } else if (isDev) {
         if (shouldLogBody) {
           appLog(`HTTP request completed
 url=${req.url}
@@ -9528,8 +9611,33 @@ duration_ms=${durationMs}`, "HTTPTraceInterceptor");
       }
       this.traceLogger.logStructured(traceLevel, isError ? "HTTP request failed" : "HTTP request completed", responseMeta, "HTTPTraceInterceptor");
     });
-    return next.handle().pipe((0, import_operators.catchError)((error) => {
-      this.appLogger.error(format("HTTP request exception\nmethod=%s url=%s\nerror=%o", req.method, req.url, error), "HTTPTraceInterceptor");
+    return next.handle().pipe((0, import_operators.tap)({
+      next: /* @__PURE__ */ __name((data) => {
+        const statusCode = res.statusCode;
+        const durationMs = Date.now() - startedAt;
+        const isError = statusCode > 400;
+        const responseData = {
+          method,
+          path,
+          url: fullUrl,
+          duration_ms: durationMs,
+          error_message: isError ? data?.message ?? "" : ""
+        };
+        if (req.body) responseData.request_body = req.body;
+        if (data) responseData.response = data;
+        const reportData = JSON.stringify(responseData);
+        if (!isDev) {
+          this.appLogger.log(reportData, {
+            paas_attributes_module: "app_server",
+            source_type: "platform"
+          }, "HTTPTraceInterceptor");
+        }
+      }, "next")
+    }), (0, import_operators.catchError)((error) => {
+      this.appLogger.error(format("HTTP request exception\nmethod=%s url=%s\nerror=%o", req.method, req.url, error), {
+        paas_attributes_module: "app_server",
+        source_type: "platform"
+      }, "HTTPTraceInterceptor");
       throw error;
     }));
   }