@mimik/sumologic-winston-logger 2.1.12 → 2.1.14

package/.claude/settings.local.json

@@ -2,7 +2,9 @@
   "permissions": {
     "allow": [
       "Bash(npx mocha:*)",
-      "Bash(done)"
+      "Bash(done)",
+      "Bash(npm test)",
+      "Bash(git status -u)"
     ]
   }
 }

package/configuration/config.js

@@ -8,12 +8,11 @@ import {
   ENV_LOCAL,
   NONE_MODE,
   SUMOLOGIC,
+  toInt,
 } from '../lib/common.js';
 import process from 'node:process';
 import { readFileSync } from 'node:fs';
 
-const DECIMAL = 10;
-
 /**
  *
  * Logger configuration.
@@ -124,8 +123,8 @@ const configuration = {
   filter: {
     file: process.env.FILTER_FILE || null,
   },
-  flushExitDelay: parseInt(process.env.FLUSH_EXIT_DELAY, DECIMAL) || 2000, // in millisecond
-  flushExitTimeout: parseInt(process.env.FLUSH_EXIT_TIMEOUT, DECIMAL) || 5000, // in millisecond
+  flushExitDelay: toInt(process.env.FLUSH_EXIT_DELAY, 2000), // in millisecond
+  flushExitTimeout: toInt(process.env.FLUSH_EXIT_TIMEOUT, 5000), // in millisecond
   noStack: process.env.NO_STACK || 'yes',
 };
 configuration.mode = checkMode(process.env.LOG_MODE) || [NONE_MODE];
@@ -142,13 +141,13 @@ if (configuration.mode.includes(AWS_KINESIS)) {
     streamNameError: process.env.KINESIS_AWS_STREAM_NAME_ERROR,
     streamNameOther: process.env.KINESIS_AWS_STREAM_NAME_OTHER,
     region: process.env.KINESIS_AWS_REGION,
-    timeout: parseInt(process.env.KINESIS_AWS_TIMEOUT, DECIMAL) || 5, // in minute
-    maxSize: parseInt(process.env.KINESIS_AWS_MAX_SIZE, DECIMAL) || 5, // in mByte
-    maxEvents: parseInt(process.env.KINESIS_AWS_MAX_EVENTS, DECIMAL) || 1000,
-    maxRetries: parseInt(process.env.KINESIS_AWS_MAX_RETRIES, DECIMAL) || 4,
+    timeout: toInt(process.env.KINESIS_AWS_TIMEOUT, 5), // in minute
+    maxSize: toInt(process.env.KINESIS_AWS_MAX_SIZE, 5), // in mByte
+    maxEvents: toInt(process.env.KINESIS_AWS_MAX_EVENTS, 1000),
+    maxRetries: toInt(process.env.KINESIS_AWS_MAX_RETRIES, 4),
     httpOptions: {
-      socketTimeout: parseInt(process.env.KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT, DECIMAL) || 5000, // in millisecond
-      connectionTimeout: parseInt(process.env.KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT, DECIMAL) || 5000, // in millisecond
+      socketTimeout: toInt(process.env.KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT, 5000), // in millisecond
+      connectionTimeout: toInt(process.env.KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT, 5000), // in millisecond
     },
   };
 
@@ -159,9 +158,9 @@ if (configuration.mode.includes(AWS_S3)) {
   configuration[AWS_S3] = {
     bucketname: process.env.S3_AWS_BUCKET_NAME,
     region: process.env.S3_AWS_REGION,
-    timeout: parseInt(process.env.S3_AWS_TIMEOUT, DECIMAL) || 5, // in minute
-    maxSize: parseInt(process.env.S3_AWS_MAX_SIZE, DECIMAL) || 5, // in mByte
-    maxEvents: parseInt(process.env.S3_AWS_MAX_EVENTS, DECIMAL) || 1000,
+    timeout: toInt(process.env.S3_AWS_TIMEOUT, 5), // in minute
+    maxSize: toInt(process.env.S3_AWS_MAX_SIZE, 5), // in mByte
+    maxEvents: toInt(process.env.S3_AWS_MAX_EVENTS, 1000),
   };
 
   if (process.env.S3_AWS_ACCESS_KEY_ID !== undefined) configuration[AWS_S3].accessKeyId = process.env.S3_AWS_ACCESS_KEY_ID;

package/lib/common.js

@@ -53,6 +53,12 @@ const safeStringify = (obj) => {
   });
 };
 
+const DECIMAL = 10;
+const toInt = (value, fallback) => {
+  const parsed = parseInt(value, DECIMAL);
+  return Number.isNaN(parsed) ? fallback : parsed;
+};
+
 export {
   ALL_MODE,
   ALL_MODES,
@@ -65,7 +71,6 @@ export {
   ERROR,
   FLUSH,
   FLUSH_EXIT,
-  TEST_FLUSH_TIMEOUT,
   INFO,
   LEVEL,
   LOG,
@@ -79,13 +84,15 @@ export {
   OTHER,
   PARTITION_KEY,
   SPLAT,
-  TEST_START_TIMEOUT,
   SUMOLOGIC,
   SYSTEM_ERROR,
   TEAPOT_ERROR,
+  TEST_FLUSH_TIMEOUT,
+  TEST_START_TIMEOUT,
   UNAUTHORIZED_ERROR,
   UNKNOWN_TYPE,
   UNKNOWN_ID,
   WARN,
   safeStringify,
+  toInt,
 };

package/lib/formatLib.js

@@ -20,7 +20,7 @@ const isReserved = (value) => {
 
 const stackInfo = format((origInfo, opts) => {
   const { env, noStack } = opts;
-  const info = origInfo;
+  const info = { ...origInfo };
 
   if (info.level === 'error') {
     info.stackInfo = parseStack(new Error());

package/lib/stackLib.js

@@ -45,8 +45,8 @@ const parseStack = (newError) => {
   stackInfo.hash = createHash('sha256')
     .update(stackInfo.stack)
     .digest('hex');
-  // this is a hash of the stacktrace for easier searching for the stacktrace
-  // security of this is not a concern since stacktrace is also sent unencrypted
+  // this is a hash of the stack trace for easier searching for the stack trace
+  // security of this is not a concern since the stack trace is also sent unencrypted
   return stackInfo;
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mimik/sumologic-winston-logger",
-  "version": "2.1.12",
+  "version": "2.1.14",
   "description": "Log wrapper for sumo, s3, kinesis and winston",
   "main": "./index.js",
   "type": "module",
@@ -32,9 +32,9 @@
   },
   "dependencies": {
     "@mimik/lib-filters": "^2.0.7",
-    "@aws-sdk/client-s3": "3.995.0",
-    "@aws-sdk/client-kinesis": "3.995.0",
-    "@smithy/node-http-handler": "4.4.10",
+    "@aws-sdk/client-s3": "3.998.0",
+    "@aws-sdk/client-kinesis": "3.998.0",
+    "@smithy/node-http-handler": "4.4.12",
     "axios": "1.13.5",
     "winston": "3.19.0",
     "winston-transport": "4.9.0"
@@ -45,7 +45,7 @@
     "@mimik/request-helper": "^2.0.5",
     "@stylistic/eslint-plugin": "5.9.0",
     "aws-sdk-client-mock": "4.1.0",
-    "c8": "10.1.3",
+    "c8": "11.0.0",
     "chai": "6.2.2",
     "eslint": "9.39.2",
     "eslint-plugin-import": "2.32.0",

package/test/logger.spec.js

@@ -183,6 +183,18 @@ describe('Sumologic transport', function SumoTests() {
     });
     sumo.log({ level: 'info', message: 'test globalThis' });
   });
+
+  it('falls back to unknownServerId when globalThis.serverId is not set', (done) => {
+    globalThis.serverType = 'mSumoRemote';
+    sumo.code = '401';
+    sumo.once('warn', (resp) => {
+      resp.data.should.have.property('serverType', 'mSumoRemote');
+      resp.data.should.have.property('serverId', 'unknownServerId@clients');
+      delete globalThis.serverType;
+      done();
+    });
+    sumo.log({ level: 'info', message: 'test serverId fallback' });
+  });
 });
 
 describe('S3 transport', function S3Tests() {