@nm-logger/logger 1.2.4 → 1.2.6

package/package.json

@@ -1 +1 @@
-{"name":"@nm-logger/logger","version":"1.2.4","description":"Express JSON logger with daily success/error/external logs, S3 append uploads, and configurable baseDir.","main":"index.js","types":"index.d.ts","license":"MIT","dependencies":{"@aws-sdk/client-s3":"^3.600.0","fs-extra":"^11.1.1"}}
+{"name":"@nm-logger/logger","version":"1.2.6","description":"Express JSON logger with daily success/error/external logs, S3 append uploads, and configurable baseDir.","main":"index.js","types":"index.d.ts","license":"MIT","dependencies":{"@aws-sdk/client-s3":"^3.600.0","fs-extra":"^11.1.1"}}

package/src/LogWriter.js

@@ -6,6 +6,7 @@ const MAP = {
   success: "daily_logs_success.json",
   error: "daily_logs_error.json",
   external: "daily_logs_external.json",
+  db_error: "daily_logs_db_error.json",
 };
 
 class LogWriter {
@@ -45,7 +46,7 @@ class LogWriter {
       type: log.type || "",
       method: log.method || "",
       error: log.error || "",
-      employee_id: log.employee_id || "",
+      status_code: log.status_code || "",
       date: log.date || formatDate(new Date()),
     });
 

package/src/Logger.js

@@ -51,7 +51,7 @@ class Logger {
   }
 
   // Build base log object for success/error logs
-  buildBaseLog(req, employee_id = "") {
+  buildBaseLog(req, employee_id = "", statusCode) {
     const url = req && req.originalUrl ? req.originalUrl : "";
     const body = (req && req.body) || {};
     const params = {
@@ -79,12 +79,14 @@ class Logger {
       method,
       error: "",
       employee_id: emp,
+      status_code:
+        typeof statusCode === "number" ? statusCode : statusCode || "",
     };
   }
 
   // ---- Error logging (Express errors) ----
-  async logError(err, req, employee_id = "") {
-    const base = this.buildBaseLog(req || {}, employee_id);
+  async logError(err, req, employee_id = "", statusCode) {
+    const base = this.buildBaseLog(req || {}, employee_id, statusCode);
 
     const log = {
       ...base,
@@ -95,8 +97,8 @@ class Logger {
   }
 
   // ---- Success logging (normal requests) ----
-  async logRequest(req, employee_id = "") {
-    const log = this.buildBaseLog(req, employee_id);
+  async logRequest(req, employee_id = "", statusCode) {
+    const log = this.buildBaseLog(req, employee_id, statusCode);
     await this.logWriter.writeLog(log, "success");
   }
 
@@ -107,8 +109,8 @@ class Logger {
     data,
     params,
     error,
-    employeeId,
-    response, // 👈 NEW: store external API response
+    response, // 👈 store external API response
+    statusCode,
   }) {
     const maskedBody = this.mask(data || {});
     const maskedParams = this.mask(params || {});
@@ -118,11 +120,11 @@ class Logger {
       url: url || "",
       body: JSON.stringify(maskedBody || {}),
       params: JSON.stringify(maskedParams || {}),
-      response: JSON.stringify(maskedResponse || {}), // 👈 NEW FIELD
+      response: JSON.stringify(maskedResponse || {}),
       type: "external_api",
       method: (method || "").toUpperCase(),
       error: error || "",
-      employee_id: employeeId || "",
+      status_code: typeof statusCode === "number" ? statusCode : (statusCode || ""),
     };
 
     await this.logWriter.writeLog(log, "external");
@@ -132,7 +134,8 @@ class Logger {
   expressMiddleware() {
     return (err, req, res, next) => {
       try {
-        this.logError(err, req).catch(() => {});
+        const statusCode = res && res.statusCode;
+        this.logError(err, req, "", statusCode).catch(() => {});
       } catch (_) {}
       next(err);
     };
@@ -144,14 +147,103 @@ class Logger {
       try {
         if (!req.__nm_logger_logged) {
           req.__nm_logger_logged = true;
-          this.logRequest(req).catch(() => {});
+          res.on("finish", () => {
+            try {
+              const statusCode = res && res.statusCode;
+              this.logRequest(req, "", statusCode).catch(() => {});
+            } catch (_) {}
+          });
         }
       } catch (_) {}
       next();
     };
   }
 
-  // ---- Axios interceptor ----
+  
+  // ---- Summary helper: filter logs by status_code ----
+  /**
+   * Get summary of logs filtered by HTTP status code.
+   *
+   * @param {Object} opt
+   * @param {string} [opt.date] - Date string in YYYY-MM-DD format (defaults to today).
+   * @param {string} [opt.category] - One of "success", "error", "external", "db_error".
+   * @param {number|string} [opt.statusCodePrefix] - If provided, matches codes starting with this (e.g. 5 => 5xx).
+   * @param {Array<number|string>} [opt.statusCodes] - If provided, matches any of these exact codes.
+   *
+   * @returns {Promise<{date:string,category:string,total_logs:number,matched_logs:number,by_status:Object,logs:Array}>}
+   */
+  async getStatusSummary(opt = {}) {
+    const category = opt.category || "error";
+    const dateStr = opt.date;
+
+    // Resolve date parts similar to LogWriter.getFilePath
+    const d = dateStr ? new Date(dateStr) : new Date();
+    const Y = String(d.getFullYear());
+    const M = String(d.getMonth() + 1).padStart(2, "0");
+    const D = String(d.getDate()).padStart(2, "0");
+
+    const MAP = {
+      success: "daily_logs_success.json",
+      error: "daily_logs_error.json",
+      external: "daily_logs_external.json",
+      db_error: "daily_logs_db_error.json",
+    };
+
+    const fileName = MAP[category] || MAP.success;
+    const filePath = path.join(this.baseDir, `${Y}/${M}/${D}/${fileName}`);
+
+    const result = {
+      date: `${Y}-${M}-${D}`,
+      category,
+      total_logs: 0,
+      matched_logs: 0,
+      by_status: {},
+      logs: [],
+    };
+
+    if (!(await fs.pathExists(filePath))) {
+      return result;
+    }
+
+    try {
+      const raw = await fs.readFile(filePath, "utf8");
+      if (!raw.trim()) return result;
+
+      const parsed = JSON.parse(raw);
+      const logs = Array.isArray(parsed.logs) ? parsed.logs : [];
+      result.total_logs = logs.length;
+
+      let filtered = logs;
+
+      if (Array.isArray(opt.statusCodes) && opt.statusCodes.length) {
+        const allowed = opt.statusCodes.map((c) => String(c));
+        filtered = filtered.filter((log) =>
+          allowed.includes(String(log.status_code || ""))
+        );
+      } else if (opt.statusCodePrefix !== undefined && opt.statusCodePrefix !== null) {
+        const prefix = String(opt.statusCodePrefix);
+        filtered = filtered.filter((log) =>
+          String(log.status_code || "").startsWith(prefix)
+        );
+      }
+
+      result.matched_logs = filtered.length;
+
+      const byStatus = {};
+      for (const log of filtered) {
+        const code = String(log.status_code || "");
+        if (!code) continue;
+        byStatus[code] = (byStatus[code] || 0) + 1;
+      }
+      result.by_status = byStatus;
+      result.logs = filtered;
+
+      return result;
+    } catch (_) {
+      return result;
+    }
+  }
+// ---- Axios interceptor ----
   attachAxiosLogger(axiosInstance) {
     if (!axiosInstance || !axiosInstance.interceptors) return;
 
@@ -167,7 +259,7 @@ class Logger {
             params: cfg.params,
             response: response.data, // 👈 log external API response body
             error: null,
-            employeeId: cfg.employee_id,
+            statusCode: response && response.status,
           }).catch(() => {});
         } catch (_) {}
 
@@ -180,6 +272,8 @@ class Logger {
           const cfg = (error && error.config) || {};
           const res = error && error.response;
 
+          const statusCode = res && res.status;
+
           this.logExternalApi({
             url: cfg && cfg.url,
             method: cfg && cfg.method,
@@ -187,7 +281,7 @@ class Logger {
             params: cfg && cfg.params,
             response: res && res.data, // 👈 log response even on error
             error: error && error.message,
-            employeeId: cfg && cfg.employee_id,
+            statusCode,
           }).catch(() => {});
         } catch (_) {}
 
@@ -195,6 +289,25 @@ class Logger {
       }
     );
   }
+
+  // ---- Database error logging ----
+  async logDbError(error, query = "", params = []) {
+    try {
+      await this.logWriter.writeLog(
+        {
+          url: "",
+          body: JSON.stringify({ query, params }),
+          params: JSON.stringify(params || []),
+          response: "",
+          type: "db_error",
+          method: "",
+          error: error && (error.message || String(error)),
+          date: new Date().toISOString(),
+        },
+        "db_error"
+      );
+    } catch (_) {}
+  }
 }
 
 module.exports = Logger;

package/src/utils.js

@@ -1,6 +1,6 @@
 exports.getDatePath=function(){const d=new Date();return{Y:d.getFullYear(),M:String(d.getMonth()+1).padStart(2,"0"),D:String(d.getDate()).padStart(2,"0")};};
 exports.formatDate=function(d){const p=n=>String(n).padStart(2,"0");return d.getFullYear()+"-"+p(d.getMonth()+1)+"-"+p(d.getDate())+" "+p(d.getHours())+":"+p(d.getMinutes())+":"+p(d.getSeconds());};
 exports.getApiType=function(url){if(!url)return"";const s=url.split("/").filter(Boolean);return s[s.length-1]||"";};
-const MK=["password","pass","token","secret","otp","auth","authorization","apikey","api_key","session","ssn"];
+const MK=["password","pass","token","secret","otp","auth","authorization","apikey","api_key","session","ssn","AccessKey"];
 exports.maskSensitive=function(v,extra){const keys=[...MK,...(extra||[])].map(x=>String(x).toLowerCase());const isMask=k=>keys.some(m=>String(k).toLowerCase().includes(m));const maskAny=val=>{if(val&&typeof val==="object"){if(Array.isArray(val))return val.map(maskAny);const o={};for(const[k,x]of Object.entries(val))o[k]=isMask(k)?"*****":maskAny(x);return o;}return val;};if(typeof v==="string"){try{return maskAny(JSON.parse(v));}catch(_){return v;}}return maskAny(v);};
 exports.streamToString=async s=>await new Promise((res,rej)=>{const c=[];s.on("data",x=>c.push(x));s.on("error",rej);s.on("end",()=>res(Buffer.concat(c).toString("utf8")));});