@nm-logger/logger 1.1.5 → 1.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nm-logger/logger",
-  "version": "1.1.5",
+  "version": "1.1.6",
   "description": "Express JSON logger with S3 upload, correlation IDs, and separate success/error/external daily logs.",
   "main": "index.js",
   "types": "index.d.ts",
@@ -21,7 +21,7 @@
   "author": "nm-logger",
   "license": "MIT",
   "dependencies": {
-    "aws-sdk": "^2.1554.0",
+    "@aws-sdk/client-s3": "^3.600.0",
     "fs-extra": "^11.1.1"
   },
   "peerDependencies": {

package/src/Logger.js

@@ -140,31 +140,34 @@ class Logger {
     return this.expressMiddleware();
   }
 
-  requestLoggerMiddleware() {
-    return (req, res, next) => {
-      try {
-        let correlationId =
-          req.headers["x-correlation-id"] ||
-          req.headers["X-Correlation-ID"] ||
-          req.correlationId;
+requestLoggerMiddleware() {
+  return (req, res, next) => {
+    try {
+      // Prevent duplicate logging
+      if (req.__nm_logger_logged) {
+        return next();
+      }
+      req.__nm_logger_logged = true;
 
-        if (!correlationId) {
-          correlationId = generateCorrelationId();
-        }
+      let correlationId =
+        req.headers["x-correlation-id"] ||
+        req.headers["X-Correlation-ID"] ||
+        req.correlationId ||
+        generateCorrelationId();
 
-        req.correlationId = correlationId;
-        res.setHeader("X-Correlation-ID", correlationId);
+      req.correlationId = correlationId;
+      res.setHeader("X-Correlation-ID", correlationId);
 
-        this.logRequest(req).catch((e) =>
-          console.error("Logger requestLoggerMiddleware error:", e)
-        );
-      } catch (e) {
-        console.error("Logger requestLoggerMiddleware outer error:", e);
-      }
+      this.logRequest(req).catch((e) =>
+        console.error("Logger requestLoggerMiddleware error:", e)
+      );
+    } catch (e) {
+      console.error("Logger requestLoggerMiddleware outer error:", e);
+    }
 
-      next();
-    };
-  }
+    next();
+  };
+}
 
   attachAxiosLogger(axiosInstance) {
     if (!axiosInstance || !axiosInstance.interceptors) {

package/src/S3Uploader.js

@@ -1,26 +1,32 @@
 const fs = require("fs");
-const AWS = require("aws-sdk");
+const {
+  S3Client,
+  PutObjectCommand
+} = require("@aws-sdk/client-s3");
 
 class S3Uploader {
   constructor(config) {
-    this.s3 = new AWS.S3({
-      accessKeyId: config.accessKeyId,
-      secretAccessKey: config.secretAccessKey,
-      region: config.region
+    this.client = new S3Client({
+      region: config.region,
+      credentials: {
+        accessKeyId: config.accessKeyId,
+        secretAccessKey: config.secretAccessKey
+      }
     });
+
     this.bucket = config.bucket;
   }
 
   async upload(filePath, s3Key) {
     const fileData = fs.readFileSync(filePath);
 
-    return this.s3
-      .putObject({
-        Bucket: this.bucket,
-        Key: s3Key,
-        Body: fileData
-      })
-      .promise();
+    const command = new PutObjectCommand({
+      Bucket: this.bucket,
+      Key: s3Key,
+      Body: fileData
+    });
+
+    await this.client.send(command);
   }
 }