@mimik/sumologic-winston-logger 2.1.9 → 2.1.11

package/README.md

@@ -11,9 +11,9 @@ The following environment variables are used to configure the logger:
 | LOG_LEVEL | Log level for the running instance | debug |  |
 | CONSOLE_LEVEL | Console log level | debug |  |
 | FILTER_FILE | Filename containing filter rules | null |  |
-| EXIT_DELAY | Delay before exiting gracefully | 2000 | in millisecond |
-| NO_STACK | Whether to include call stacks in all log messages | no | expected: yes/no |
-| LOG_MODE | Comma-separated list defining the log mode/backends | sumologic | enum: awsS3, awsKinesis, sumologic, all, none |
+| EXIT_DELAY | Delay for flushing the transports and exiting | 2000 | in millisecond |
+| NO_STACK | Whether to include call stacks in all log messages | yes | expected: yes/no |
+| LOG_MODE | Comma-separated list defining the log mode/backends | none | enum: awsS3, awsKinesis, sumologic, all, none |
 
 If `LOG_MODE` includes `sumologic`, the following environment variables are required:
 
@@ -30,14 +30,14 @@ If `LOG_MODE` includes `awsKinesis`, the following environment variables are req
 | KINESIS_AWS_STREAM_NAME_ERROR | Stream name for `error` level logs |  |  |
 | KINESIS_AWS_STREAM_NAME_OTHER | Stream name for any other level |  |  |
 | KINESIS_AWS_REGION | AWS region of the stream |  |  |
-| KINESIS_AWS_TIMEOUT | Maximum time before flushing | 1000 | in millisecond |
-| KINESIS_AWS_MAX_SIZE | Maximum accumulated log size before flushing | 5 | in MB |
-| KINESIS_AWS_MAX_EVENTS | Maximum number of accumulated logs before flushing | 1000 |  |
-| KINESIS_AWS_MAX_RETRIES | Maximum connection retries | 4 |  |
+| KINESIS_AWS_TIMEOUT | Max time before sending events to Kinesis | 5 | in minute |
+| KINESIS_AWS_MAX_SIZE | Max size of the data before sending to Kinesis | 5 | in MB |
+| KINESIS_AWS_MAX_EVENTS | Max number of events before sending to Kinesis | 1000 |  |
+| KINESIS_AWS_MAX_RETRIES | Max retries to connect to Kinesis | 4 |  |
 | KINESIS_AWS_ACCESS_KEY_ID | AWS access key ID |  |  |
 | KINESIS_AWS_SECRET_ACCESS_KEY | AWS secret access key |  |  |
-| KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT | HTTP handler socket timeout | 5000 | in millisecond |
-| KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT | HTTP handler connection timeout | 5000 | in millisecond |
+| KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT | Socket timeout for the http handler | 5000 | in millisecond |
+| KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT | Connection timeout for the http handler | 5000 | in millisecond |
 
 If `LOG_MODE` includes `awsS3`, the following environment variables are required:
 
@@ -45,9 +45,9 @@ If `LOG_MODE` includes `awsS3`, the following environment variables are required
 | ----------------- | ----------- | ------- | -------- |
 | S3_AWS_BUCKET_NAME | S3 bucket name for storing logs |  |  |
 | S3_AWS_REGION | AWS region of the bucket |  |  |
-| S3_AWS_TIMEOUT | Maximum time before flushing | 5 | in minute |
-| S3_AWS_MAX_SIZE | Maximum accumulated log size before flushing | 5 | in MB |
-| S3_AWS_MAX_EVENTS | Maximum number of accumulated logs before flushing | 1000 |  |
+| S3_AWS_TIMEOUT | Max time before sending events to S3 | 5 | in minute |
+| S3_AWS_MAX_SIZE | Max size of the data before sending to S3 | 5 | in MB |
+| S3_AWS_MAX_EVENTS | Max number of events before sending to S3 | 1000 |  |
 | S3_AWS_ACCESS_KEY_ID | AWS access key ID |  |  |
 | S3_AWS_SECRET_ACCESS_KEY | AWS secret access key |  |  |
 
@@ -66,10 +66,9 @@ If `global.serverId` is set, it overrides `SERVER_ID`.
 
 Sumologic-Winston-Logger is a log wrapper that can write to multiple logging services.
 Currently, Winston, SumoLogic, AWS Kinesis and AWS S3 are supported.
-The package also adds strackTrace info.
-StackTrace information gets tacked on to Sumologic calls, In addition and method, module, and line info or
-concatenated is added to the end of winston lines.  **Line formatting is not currently configrable.**
-The package also allow some level of filtering.
+The package also adds stackTrace info.
+StackTrace information is included in all log entries. For error-level logs, the full stack trace is added. For other log levels, method name, file name, and line number are appended.  **Line formatting is not currently configurable.**
+The package also allows some level of filtering.
 
 ## Motivation ##
 To centralize logging in a single npm node package
@@ -80,12 +79,12 @@ The logger is discoverable on npmjs.org.
 
 To install:
 ```
-npm install sumologic-winston-logger --save
+npm install @mimik/sumologic-winston-logger --save
 ```
 
 ## Configuration - Details ##
 
-The following sections decribe the details of each of the environment variables listed above.
+The following sections describe the details of each of the environment variables listed above.
 
 ### `LOG_LEVEL` ###
 Log levels supported inclusively according to the following list, as borrowed from winston:
@@ -106,14 +105,14 @@ Thus, if one declares a log level of silly, the levels error, warn, info, debug,
 ### `SUMO_LOGIC_ENDPOINT` ###
 The endpoint defined in SumoLogic to where log information will be sent. See the section, _Finding SumoLogic endpoint and collector code_, below to find the value to assign to this environment variable.
 
-|Example to set the environment varible `SUMO_LOGIC_ENDPOINT`|
+|Example to set the environment variable `SUMO_LOGIC_ENDPOINT`|
 |---|
 |`export SUMO_LOGIC_ENDPOINT=https://endpoint1.collection.us2.sumologic.com/receiver/v1/http/`|
 
 ### `SUMO_LOGIC_COLLECTOR_CODE` ###
 The collector code as defined in SumoLogic. The Collector Code is the Base64 string that appears after the last slash in the URL defined in the SumoLogic Control Panel.
 
-|Example to set the environment varible `SUMO_LOGIC_COLLECTOR_CODE`|
+|Example to set the environment variable `SUMO_LOGIC_COLLECTOR_CODE`|
 |---|
 |`export SUMO_LOGIC_COLLECTOR_CODE=AhfheisdcOllectorCodeInSumoLogicuZw==`|
 
@@ -145,20 +144,20 @@ The collector code is the Base64 encrypted part of the URL that follows the last
 ### `FILTER_FILE` ###
 
 FILTER_FILE is the location where the definition of the filtering configuration is. The location has to be a full file name.
-When the environment (NODE_ENV) in which the logger is used is `prod` or `production`, the content of the log will be filtered according to the log filtering configuration included in the file refered by the FILTER_FILE valiable. 
+When the environment (NODE_ENV) in which the logger is used is `prod` or `production`, the content of the log will be filtered according to the log filtering configuration included in the file referred by the FILTER_FILE variable. 
 The filter will replace values of the designated property names to '-----'.
 
 ## Sample Use ##
 
-The intent of this package is to support all the behavior common to console.log while also including support of multiple arguements and all data types.
+The intent of this package is to support all the behavior common to console.log while also including support of multiple arguments and all data types.
 
 Formatting console logs is left to winston, except that some stackTrace info is appended to each line.
 
 Formatting of SumoLogic logs is handled by this module in the following ways:
 
-* only the first is use for message
+* only the first argument is used as the message
 * only one object can be passed as parameter
-* structured stackTrace info is added to every log except is `NO_STACK` is set the 'yes'
+* structured stackTrace info is added to every log except when NO_STACK is set to 'yes'
 * if the last parameter is a string it will be considered as a `correlationId`
 
 
@@ -166,7 +165,7 @@ Formatting of SumoLogic logs is handled by this module in the following ways:
 Listing 2 below show you how to declare a logger to run under ECMAScript 6 and then log using the various log levels supported by the Sumologic-Winston-Logger.
 
 ``` javascript
-import logger from 'sumologic-winston-logger';
+import logger from '@mimik/sumologic-winston-logger';
 
 logger.log('debug', 'this is a debug statement using log');
 logger.debug({ message: 'this is a debug statement using an object'});

package/README_Supplement.md

@@ -3,10 +3,9 @@
 
 Sumologic-Winston-Logger is a log wrapper that can write to multiple logging services.
 Currently, Winston, SumoLogic, AWS Kinesis and AWS S3 are supported.
-The package also adds strackTrace info.
-StackTrace information gets tacked on to Sumologic calls, In addition and method, module, and line info or
-concatenated is added to the end of winston lines.  **Line formatting is not currently configrable.**
-The package also allow some level of filtering.
+The package also adds stackTrace info.
+StackTrace information is included in all log entries. For error-level logs, the full stack trace is added. For other log levels, method name, file name, and line number are appended.  **Line formatting is not currently configurable.**
+The package also allows some level of filtering.
 
 ## Motivation ##
 To centralize logging in a single npm node package
@@ -17,12 +16,12 @@ The logger is discoverable on npmjs.org.
 
 To install:
 ```
-npm install sumologic-winston-logger --save
+npm install @mimik/sumologic-winston-logger --save
 ```
 
 ## Configuration - Details ##
 
-The following sections decribe the details of each of the environment variables listed above.
+The following sections describe the details of each of the environment variables listed above.
 
 ### `LOG_LEVEL` ###
 Log levels supported inclusively according to the following list, as borrowed from winston:
@@ -43,14 +42,14 @@ Thus, if one declares a log level of silly, the levels error, warn, info, debug,
 ### `SUMO_LOGIC_ENDPOINT` ###
 The endpoint defined in SumoLogic to where log information will be sent. See the section, _Finding SumoLogic endpoint and collector code_, below to find the value to assign to this environment variable.
 
-|Example to set the environment varible `SUMO_LOGIC_ENDPOINT`|
+|Example to set the environment variable `SUMO_LOGIC_ENDPOINT`|
 |---|
 |`export SUMO_LOGIC_ENDPOINT=https://endpoint1.collection.us2.sumologic.com/receiver/v1/http/`|
 
 ### `SUMO_LOGIC_COLLECTOR_CODE` ###
 The collector code as defined in SumoLogic. The Collector Code is the Base64 string that appears after the last slash in the URL defined in the SumoLogic Control Panel.
 
-|Example to set the environment varible `SUMO_LOGIC_COLLECTOR_CODE`|
+|Example to set the environment variable `SUMO_LOGIC_COLLECTOR_CODE`|
 |---|
 |`export SUMO_LOGIC_COLLECTOR_CODE=AhfheisdcOllectorCodeInSumoLogicuZw==`|
 
@@ -82,20 +81,20 @@ The collector code is the Base64 encrypted part of the URL that follows the last
 ### `FILTER_FILE` ###
 
 FILTER_FILE is the location where the definition of the filtering configuration is. The location has to be a full file name.
-When the environment (NODE_ENV) in which the logger is used is `prod` or `production`, the content of the log will be filtered according to the log filtering configuration included in the file refered by the FILTER_FILE valiable. 
+When the environment (NODE_ENV) in which the logger is used is `prod` or `production`, the content of the log will be filtered according to the log filtering configuration included in the file referred by the FILTER_FILE variable. 
 The filter will replace values of the designated property names to '-----'.
 
 ## Sample Use ##
 
-The intent of this package is to support all the behavior common to console.log while also including support of multiple arguements and all data types.
+The intent of this package is to support all the behavior common to console.log while also including support of multiple arguments and all data types.
 
 Formatting console logs is left to winston, except that some stackTrace info is appended to each line.
 
 Formatting of SumoLogic logs is handled by this module in the following ways:
 
-* only the first is use for message
+* only the first argument is used as the message
 * only one object can be passed as parameter
-* structured stackTrace info is added to every log except is `NO_STACK` is set the 'yes'
+* structured stackTrace info is added to every log except when NO_STACK is set to 'yes'
 * if the last parameter is a string it will be considered as a `correlationId`
 
 
@@ -103,7 +102,7 @@ Formatting of SumoLogic logs is handled by this module in the following ways:
 Listing 2 below show you how to declare a logger to run under ECMAScript 6 and then log using the various log levels supported by the Sumologic-Winston-Logger.
 
 ``` javascript
-import logger from 'sumologic-winston-logger';
+import logger from '@mimik/sumologic-winston-logger';
 
 logger.log('debug', 'this is a debug statement using log');
 logger.debug({ message: 'this is a debug statement using an object'});

package/configuration/config.js

@@ -1,19 +1,16 @@
-/* eslint no-process-env: "off" */
+/* eslint-disable no-process-env, no-magic-numbers */
 import {
   ALL_MODE,
   ALL_MODES,
   AWS_KINESIS,
   AWS_S3,
-  DEFAULT,
+  DEBUG_LEVEL,
+  ENV_LOCAL,
   NONE_MODE,
   SUMOLOGIC,
 } from '../lib/common.js';
-import difference from 'lodash.difference';
-import isNil from 'lodash.isnil';
-import isUndefined from 'lodash.isundefined';
 import process from 'node:process';
 import { readFileSync } from 'node:fs';
-import split from 'lodash.split';
 
 const DECIMAL = 10;
 
@@ -33,9 +30,9 @@ const DECIMAL = 10;
  * | LOG_LEVEL | Log level for the running instance | debug |  |
  * | CONSOLE_LEVEL | Console log level | debug |  |
  * | FILTER_FILE | Filename containing filter rules | null |  |
- * | EXIT_DELAY | Delay before exiting gracefully | 2000 | in millisecond |
- * | NO_STACK | Whether to include call stacks in all log messages | no | expected: yes/no |
- * | LOG_MODE | Comma-separated list defining the log mode/backends | sumologic | enum: awsS3, awsKinesis, sumologic, all, none |
+ * | EXIT_DELAY | Delay for flushing the transports and exiting | 2000 | in millisecond |
+ * | NO_STACK | Whether to include call stacks in all log messages | yes | expected: yes/no |
+ * | LOG_MODE | Comma-separated list defining the log mode/backends | none | enum: awsS3, awsKinesis, sumologic, all, none |
  *
  * If `LOG_MODE` includes `sumologic`, the following environment variables are required:
  *
@@ -52,14 +49,14 @@ const DECIMAL = 10;
  * | KINESIS_AWS_STREAM_NAME_ERROR | Stream name for `error` level logs |  |  |
  * | KINESIS_AWS_STREAM_NAME_OTHER | Stream name for any other level |  |  |
  * | KINESIS_AWS_REGION | AWS region of the stream |  |  |
- * | KINESIS_AWS_TIMEOUT | Maximum time before flushing | 1000 | in millisecond |
- * | KINESIS_AWS_MAX_SIZE | Maximum accumulated log size before flushing | 5 | in MB |
- * | KINESIS_AWS_MAX_EVENTS | Maximum number of accumulated logs before flushing | 1000 |  |
- * | KINESIS_AWS_MAX_RETRIES | Maximum connection retries | 4 |  |
+ * | KINESIS_AWS_TIMEOUT | Max time before sending events to Kinesis | 5 | in minute |
+ * | KINESIS_AWS_MAX_SIZE | Max size of the data before sending to Kinesis | 5 | in MB |
+ * | KINESIS_AWS_MAX_EVENTS | Max number of events before sending to Kinesis | 1000 |  |
+ * | KINESIS_AWS_MAX_RETRIES | Max retries to connect to Kinesis | 4 |  |
  * | KINESIS_AWS_ACCESS_KEY_ID | AWS access key ID |  |  |
  * | KINESIS_AWS_SECRET_ACCESS_KEY | AWS secret access key |  |  |
- * | KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT | HTTP handler socket timeout | 5000 | in millisecond |
- * | KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT | HTTP handler connection timeout | 5000 | in millisecond |
+ * | KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT | Socket timeout for the http handler | 5000 | in millisecond |
+ * | KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT | Connection timeout for the http handler | 5000 | in millisecond |
  *
  * If `LOG_MODE` includes `awsS3`, the following environment variables are required:
  *
@@ -67,9 +64,9 @@ const DECIMAL = 10;
  * | ----------------- | ----------- | ------- | -------- |
  * | S3_AWS_BUCKET_NAME | S3 bucket name for storing logs |  |  |
  * | S3_AWS_REGION | AWS region of the bucket |  |  |
- * | S3_AWS_TIMEOUT | Maximum time before flushing | 5 | in minute |
- * | S3_AWS_MAX_SIZE | Maximum accumulated log size before flushing | 5 | in MB |
- * | S3_AWS_MAX_EVENTS | Maximum number of accumulated logs before flushing | 1000 |  |
+ * | S3_AWS_TIMEOUT | Max time before sending events to S3 | 5 | in minute |
+ * | S3_AWS_MAX_SIZE | Max size of the data before sending to S3 | 5 | in MB |
+ * | S3_AWS_MAX_EVENTS | Max number of events before sending to S3 | 1000 |  |
  * | S3_AWS_ACCESS_KEY_ID | AWS access key ID |  |  |
  * | S3_AWS_SECRET_ACCESS_KEY | AWS secret access key |  |  |
  *
@@ -90,12 +87,12 @@ const checkConfig = (config) => {
       if (typeof node[prop] === 'object' && node[prop]) {
         traverseNodeSync(node[prop], `${path}.${prop}`);
       }
-      else if (isUndefined(node[prop])) errs.push(`${path}.${prop}`);
+      else if (node[prop] === undefined) errs.push(`${path}.${prop}`);
     });
   };
 
   traverseNodeSync(config, 'configuration');
-  if (errs.length > 1) {
+  if (errs.length > 0) {
     throw new Error(`Missing values for ${errs}`);
   }
 };
@@ -104,9 +101,9 @@ const checkMode = (mode) => {
   let logMode = null;
 
   if (mode) {
-    logMode = split(mode.trim(), /\s*,\s*/u);
+    logMode = mode.trim().split(/\s*,\s*/u);
     if (logMode.length === 0) throw new Error('Invalid LOG_MODE: cannot be an empty array');
-    if (difference(logMode, ALL_MODES).length !== 0) throw new Error(`Invalid items in LOG_MODE: ${mode}`);
+    if (logMode.some(item => !ALL_MODES.includes(item))) throw new Error(`Invalid items in LOG_MODE: ${mode}`);
     if (logMode.includes(NONE_MODE) && logMode.length !== 1) throw new Error(`Cannot have multiple modes when ${NONE_MODE} is selected`);
     if (logMode.includes(ALL_MODE)) logMode = [SUMOLOGIC, AWS_S3]; // legacy support
   }
@@ -118,18 +115,18 @@ const configuration = {
     type: process.env.SERVER_TYPE || null,
     id: process.env.SERVER_ID || null,
   },
-  env: process.env.NODE_ENV || DEFAULT.ENV,
+  env: process.env.NODE_ENV || ENV_LOCAL,
   level: {
-    log: process.env.LOG_LEVEL || DEFAULT.LEVEL,
-    console: process.env.CONSOLE_LEVEL || DEFAULT.LEVEL,
+    log: process.env.LOG_LEVEL || DEBUG_LEVEL,
+    console: process.env.CONSOLE_LEVEL || DEBUG_LEVEL,
   },
   filter: {
-    file: process.env.FILTER_FILE || DEFAULT.FILTER_FILE,
+    file: process.env.FILTER_FILE || null,
   },
-  exitDelay: parseInt(process.env.EXIT_DELAY, DECIMAL) || DEFAULT.EXIT_DELAY, // in millisecond
-  noStack: process.env.NO_STACK || DEFAULT.NO_STACK,
+  exitDelay: parseInt(process.env.EXIT_DELAY, DECIMAL) || 2000, // in millisecond
+  noStack: process.env.NO_STACK || 'yes',
 };
-configuration.mode = checkMode(process.env.LOG_MODE) || DEFAULT.MODE;
+configuration.mode = checkMode(process.env.LOG_MODE) || [NONE_MODE];
 
 if (configuration.mode.includes(SUMOLOGIC)) {
   configuration[SUMOLOGIC] = {
@@ -143,30 +140,30 @@ 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) || DEFAULT.KINESIS_TIMEOUT, // in millisecond
-    maxSize: parseInt(process.env.KINESIS_AWS_MAX_SIZE, DECIMAL) || DEFAULT.KINESIS_MAX_SIZE, // in mByte
-    maxEvents: parseInt(process.env.KINESIS_AWS_MAX_EVENTS, DECIMAL) || DEFAULT.KINESIS_MAX_EVENTS,
-    maxRetries: parseInt(process.env.KINESIS_AWS_MAX_RETRIES, DECIMAL) || DEFAULT.KINESIS_MAX_RETRIES,
+    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,
     httpOptions: {
-      socketTimeout: parseInt(process.env.KINESIS_AWS_HTTP_OPTIONS_SOCKET_TIMEOUT, DECIMAL) || DEFAULT.KINESIS_HTTP_OPTIONS_SOCKET_TIMEOUT,
-      connectionTimeout: parseInt(process.env.KINESIS_AWS_HTTP_OPTIONS_CONNECTION_TIMEOUT, DECIMAL) || DEFAULT.KINESIS_HTTP_OPTIONS_CONNECTION_TIMEOUT,
+      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
     },
   };
 
-  if (!isNil(process.env.KINESIS_AWS_ACCESS_KEY_ID)) configuration[AWS_KINESIS].accessKeyId = process.env.KINESIS_AWS_ACCESS_KEY_ID;
-  if (!isNil(process.env.KINESIS_AWS_SECRET_ACCESS_KEY)) configuration[AWS_KINESIS].secretAccessKey = process.env.KINESIS_AWS_SECRET_ACCESS_KEY;
+  if (process.env.KINESIS_AWS_ACCESS_KEY_ID !== undefined) configuration[AWS_KINESIS].accessKeyId = process.env.KINESIS_AWS_ACCESS_KEY_ID;
+  if (process.env.KINESIS_AWS_SECRET_ACCESS_KEY !== undefined) configuration[AWS_KINESIS].secretAccessKey = process.env.KINESIS_AWS_SECRET_ACCESS_KEY;
 }
 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) || DEFAULT.S3_TIMEOUT, // in minute
-    maxSize: parseInt(process.env.S3_AWS_MAX_SIZE, DECIMAL) || DEFAULT.S3_MAX_SIZE, // in mByte
-    maxEvents: parseInt(process.env.S3_AWS_MAX_EVENTS, DECIMAL) || DEFAULT.S3_MAX_EVENTS,
+    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,
   };
 
-  if (!isNil(process.env.S3_AWS_ACCESS_KEY_ID)) configuration[AWS_S3].accessKeyId = process.env.S3_AWS_ACCESS_KEY_ID;
-  if (!isNil(process.env.S3_AWS_SECRET_ACCESS_KEY)) configuration[AWS_S3].secretAccessKey = process.env.S3_AWS_SECRET_ACCESS_KEY;
+  if (process.env.S3_AWS_ACCESS_KEY_ID !== undefined) configuration[AWS_S3].accessKeyId = process.env.S3_AWS_ACCESS_KEY_ID;
+  if (process.env.S3_AWS_SECRET_ACCESS_KEY !== undefined) configuration[AWS_S3].secretAccessKey = process.env.S3_AWS_SECRET_ACCESS_KEY;
 }
 const { filter } = configuration;
 let filterConfig = [];

package/eslint.config.js

@@ -30,10 +30,8 @@ export default [
       ecmaVersion: ECMA_VERSION,
       globals: {
         ...globals.nodeBuiltin,
-        console: 'readonly',
         describe: 'readonly',
         it: 'readonly',
-        require: 'readonly',
       },
       sourceType: 'module',
     },
@@ -41,6 +39,7 @@ export default [
       '@stylistic/brace-style': ['warn', 'stroustrup', { allowSingleLine: true }],
       '@stylistic/line-comment-position': ['off'],
       '@stylistic/max-len': ['warn', MAX_LENGTH_LINE, { ignoreComments: true, ignoreStrings: true, ignoreRegExpLiterals: true }],
+      '@stylistic/quotes': ['warn', 'single'],
       '@stylistic/semi': ['error', 'always'],
       'capitalized-comments': ['off'],
       'complexity': ['error', MAX_COMPLEXITY],
@@ -51,6 +50,7 @@ export default [
       'init-declarations': ['off'],
       'linebreak-style': ['off'],
       'max-depth': ['error', MAX_DEPTH],
+      'max-len': 'off',
       'max-lines': ['warn', { max: MAX_LINES_IN_FILES, skipComments: true, skipBlankLines: true }],
       'max-lines-per-function': ['warn', { max: MAX_LINES_IN_FUNCTION, skipComments: true, skipBlankLines: true }],
       'max-params': ['error', MAX_FUNCTION_PARAMETERS],
@@ -63,7 +63,7 @@ export default [
       'no-undefined': ['off'],
       'one-var': ['error', 'never'],
       'processDoc/validate-document-env': ['error'],
-      'quotes': ['warn', 'single'],
+      'quotes': 'off',
       'sort-imports': ['error', { allowSeparatedGroups: true }],
       'sort-keys': ['error', 'asc', { caseSensitive: true, minKeys: MIN_KEYS_IN_OBJECT, natural: false, allowLineSeparatedGroups: true }],
     },

package/index.js

@@ -118,7 +118,7 @@ logger.flushAndExit = (code) => {
     if (config.mode.includes(NONE_MODE)) return process.exit(code);
     if (awsS3) {
       awsS3.flush(FLUSH_EXIT);
-      awsS3.on(FLUSH_EXIT, () => {
+      awsS3.once(FLUSH_EXIT, () => {
         if ((!sumo || sumoDone) && (!awsKinesis || awsKinesisDone)) return process.exit(code);
         awsS3Done = true;
         return null;
@@ -126,7 +126,7 @@ logger.flushAndExit = (code) => {
     }
     if (sumo) {
       sumo.flush(FLUSH_EXIT);
-      sumo.on(FLUSH_EXIT, () => {
+      sumo.once(FLUSH_EXIT, () => {
         if ((!awsS3 || awsS3Done) && (!awsKinesis || awsKinesisDone)) return process.exit(code);
         sumoDone = true;
         return null;
@@ -134,7 +134,7 @@ logger.flushAndExit = (code) => {
     }
     if (awsKinesis) {
       awsKinesis.flush(FLUSH_EXIT);
-      awsKinesis.on(FLUSH_EXIT, () => {
+      awsKinesis.once(FLUSH_EXIT, () => {
         if ((!awsS3 || awsS3Done) && (!sumo || sumoDone)) return process.exit(code);
         awsKinesisDone = true;
         return null;
@@ -155,23 +155,23 @@ logger.flush = () => {
     if (config.mode.includes(NONE_MODE)) return null;
     if (awsS3) {
       awsS3.flush(FLUSH);
-      awsS3.on(FLUSH, () => {
+      awsS3.once(FLUSH, () => {
         if ((!sumo || sumoDone) && (!awsKinesis || awsKinesisDone)) return null;
         awsS3Done = true;
         return null;
       });
     }
     if (sumo) {
-      sumo.flush();
-      sumo.on(FLUSH, () => {
+      sumo.flush(FLUSH);
+      sumo.once(FLUSH, () => {
         if ((!awsS3 || awsS3Done) && (!awsKinesis || awsKinesisDone)) return null;
         sumoDone = true;
         return null;
       });
     }
     if (awsKinesis) {
-      awsKinesis.flush();
-      awsKinesis.on(FLUSH, () => {
+      awsKinesis.flush(FLUSH);
+      awsKinesis.once(FLUSH, () => {
         if ((!awsS3 || awsS3Done) && (!sumo || sumoDone)) return null;
         awsKinesisDone = true;
         return null;

package/lib/awsKinesisTransport.js

@@ -13,6 +13,7 @@ import {
   UNKNOWN_ID,
   UNKNOWN_TYPE,
   WARN,
+  safeStringify,
 } from './common.js';
 import {
   KinesisClient,
@@ -24,7 +25,6 @@ import {
 } from 'node:timers';
 import { Buffer } from 'node:buffer';
 import { NodeHttpHandler } from '@smithy/node-http-handler';
-import Promise from 'bluebird';
 import Transport from 'winston-transport';
 
 const RANDOM_MIN = 0;
@@ -101,11 +101,11 @@ export default class AwsKinesis extends Transport {
   }
 
   flushEvent() {
-    return Promise.map(Object.keys(events), (level) => {
+    return Promise.all(Object.keys(events).map((level) => {
       if (events[level].Records.length) {
         return this.put(events[level].Records, level)
           .then(() => {
-            events[level].data = []; // we may lose some logs due to concurrency issues
+            events[level].Records = []; // we may lose some logs due to concurrency issues
             events[level].size = 0;
           })
           .catch(err => this.emit(WARN, {
@@ -115,17 +115,16 @@ export default class AwsKinesis extends Transport {
           }));
       }
       return Promise.resolve();
-    })
+    }))
       .then(() => this.emit(LOG, { message: `logs sent to ${AWS_KINESIS}` }));
   }
 
   log(info, callback) {
-    const messageInfo = info[MESSAGE];
-    const infoSize = messageInfo.length;
+    const infoSize = Buffer.byteLength(info[MESSAGE]);
     let { level } = info;
     const { serverType } = globalThis;
     let { serverId } = globalThis;
-    const data = JSON.parse(messageInfo);
+    const data = { ...info };
 
     if (serverType) {
       if (!serverId) serverId = `${UNKNOWN_ID}${CLIENTS}`;
@@ -144,7 +143,7 @@ export default class AwsKinesis extends Transport {
       levelData = events[level];
     }
     levelData.size += infoSize;
-    levelData.Records.push({ Data: Buffer.from(JSON.stringify(data)), PartitionKey: `${PARTITION_KEY}-${randomInt(RANDOM_MIN, RANDOM_LIMIT)}` });
+    levelData.Records.push({ Data: Buffer.from(safeStringify(data)), PartitionKey: `${PARTITION_KEY}-${randomInt(RANDOM_MIN, RANDOM_LIMIT)}` });
     if (levelData.Records.length >= this.maxEvents || levelData.size >= this.maxSize) {
       this.send(levelData.Records, level);
       levelData.Records = [];

package/lib/awsS3Transport.js

@@ -9,6 +9,7 @@ import {
   UNKNOWN_ID,
   UNKNOWN_TYPE,
   WARN,
+  safeStringify,
 } from './common.js';
 import {
   PutObjectCommand,
@@ -18,7 +19,6 @@ import {
   setImmediate,
   setInterval,
 } from 'node:timers';
-import Promise from 'bluebird';
 import Transport from 'winston-transport';
 
 const events = {};
@@ -89,7 +89,7 @@ export default class AwsS3 extends Transport {
     const errors = [];
     let count = 0;
 
-    return Promise.map(Object.keys(data), sType => Promise.each(Object.keys(data[sType]), (sId) => {
+    return Promise.all(Object.keys(data).map(sType => Object.keys(data[sType]).reduce((chain, sId) => chain.then(() => {
       const command = new PutObjectCommand({
         Bucket: this.bucketname,
         Key: `${lvl}/${sType}/${sId}/${date.getFullYear()}/${date.getMonth() + 1}/${date.getDate()}/${date.toISOString()}.json`,
@@ -104,7 +104,7 @@ export default class AwsS3 extends Transport {
           data: data[sType][sId],
           error: { message: err.message, statusCode: err.statusCode || SYSTEM_ERROR, details: err },
         }));
-    }))
+    }), Promise.resolve())))
       .then(() => ({ count, errors }));
   }
 
@@ -118,7 +118,7 @@ export default class AwsS3 extends Transport {
         if (count === 0) {
           return this.emit(WARN, {
             errors,
-            nblogs: errorCount,
+            nbLogs: errorCount,
             nbLogsSent: 0,
             message: `could not log to ${AWS_S3}`,
           });
@@ -133,7 +133,7 @@ export default class AwsS3 extends Transport {
   }
 
   flushEvent() {
-    return Promise.map(Object.keys(events), (level) => {
+    return Promise.all(Object.keys(events).map((level) => {
       if (events[level].data.length) {
         return this.put(events[level].data, level, new Date())
           .then(() => {
@@ -143,12 +143,12 @@ export default class AwsS3 extends Transport {
           .catch(err => this.emit(WARN, { error: err.message, message: `could not log to ${AWS_S3}` }));
       }
       return Promise.resolve();
-    })
+    }))
       .then(() => this.emit(LOG, { message: `logs sent to ${AWS_S3}` }));
   }
 
   flushTypeEvent() {
-    return Promise.map(Object.keys(typeEvents), (level) => {
+    return Promise.all(Object.keys(typeEvents).map((level) => {
       if (Object.keys(typeEvents[level].data).length) {
         return this.putRemote(typeEvents[level].data, level, new Date())
           .then((result) => {
@@ -162,7 +162,7 @@ export default class AwsS3 extends Transport {
             if (count === 0) {
               return this.emit(WARN, {
                 errors,
-                nblogs: errorCount,
+                nbLogs: errorCount,
                 nbLogsSent: 0,
                 message: `could not log to ${AWS_S3}`,
               });
@@ -176,16 +176,15 @@ export default class AwsS3 extends Transport {
           });
       }
       return Promise.resolve();
-    });
+    }));
   }
 
   log(info, callback) {
-    const messageInfo = info[MESSAGE];
-    const infoSize = messageInfo.length;
+    const infoSize = Buffer.byteLength(info[MESSAGE]);
     const { level } = info;
     const { serverType } = globalThis;
     let { serverId } = globalThis;
-    const data = JSON.parse(messageInfo);
+    const data = { ...info };
 
     if (serverType) {
       if (!serverId) serverId = `${UNKNOWN_ID}${CLIENTS}`;
@@ -211,7 +210,7 @@ export default class AwsS3 extends Transport {
       }
       typeLevelData.size += infoSize;
       typeLevelData.nbEvents += 1;
-      serverData.push(data);
+      serverData.push(safeStringify(data));
       if (typeLevelData.nbEvents >= this.maxEvents || typeLevelData.size >= this.maxSize) {
         this.sendRemote(typeLevelData.data, level, new Date());
         typeLevelData.data = {};
@@ -229,7 +228,7 @@ export default class AwsS3 extends Transport {
         levelData = events[level];
       }
       levelData.size += infoSize;
-      levelData.data.push(data);
+      levelData.data.push(safeStringify(data));
       if (levelData.data.length >= this.maxEvents || levelData.size >= this.maxSize) {
         this.send(levelData.data, level, new Date());
         levelData.data = [];