@mhmdhammoud/meritt-utils 1.4.1 → 1.4.2

package/ReleaseNotes.md

@@ -1,5 +1,9 @@
 # Changes
 
+## Version 1.4.1
+
+- Json stringify the log message if its being pushed to Kibana
+
 ## Version 1.4.0
 
 - Changed winston logger to Pino logger to improve performance and reduce memory usage

package/dist/lib/logger.d.ts

@@ -1,4 +1,5 @@
-import { ElasticConfig, LOG_LEVEL, LogEvent } from '../types';
+import { Options as ElasticConfig } from 'pino-elasticsearch';
+import { LOG_LEVEL, LogEvent } from '../types';
 /**
  * Checks if a given log level is valid.
  * @param level - The log level to check.

package/dist/lib/logger.js

@@ -52,7 +52,8 @@ function getLogger(elasticConfig) {
                         username: process.env.ELASTICSEARCH_USERNAME,
                         password: process.env.ELASTICSEARCH_PASSWORD,
                     },
-                    flushInterval: 10000,
+                    flushInterval: 1000,
+                    'flush-bytes': 1000,
                 };
                 if (elasticConfig) {
                     Object.assign(esConfig, elasticConfig);
@@ -63,7 +64,7 @@ function getLogger(elasticConfig) {
             else {
                 transports.push(pino_1.pino.destination({
                     minLength: 1024,
-                    sync: true,
+                    sync: false,
                 }));
             }
             pinoLogger = (0, pino_1.pino)({
@@ -96,10 +97,18 @@ class Logger {
         this._logger = getLogger(elasticConfig);
     }
     log(logLevel, logEvent, ...args) {
+        let detail;
+        if (process.env.NODE_ENV === 'local' || process.env.NODE_ENV === 'test') {
+            detail = args;
+        }
+        else {
+            //@ts-ignore
+            detail = JSON.stringify(...args);
+        }
         this._logger[logLevel]({
             component: this._name,
             ...logEvent,
-            detail: args,
+            detail,
         });
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@mhmdhammoud/meritt-utils",
-	"version": "1.4.1",
+	"version": "1.4.2",
 	"description": "",
 	"main": "./dist/index.js",
 	"private": false,

package/src/lib/logger.ts

@@ -1,7 +1,7 @@
 import {Logger as PinoLogger, pino, stdTimeFunctions} from 'pino'
 import * as dotenv from 'dotenv'
-import pinoElastic from 'pino-elasticsearch'
-import {ElasticConfig, LOG_LEVEL, LogEvent} from '../types'
+import pinoElastic, {Options as ElasticConfig} from 'pino-elasticsearch'
+import {LOG_LEVEL, LogEvent} from '../types'
 
 dotenv.config()
 
@@ -27,21 +27,25 @@ function getLogger(elasticConfig?: ElasticConfig): PinoLogger {
 						username: process.env.ELASTICSEARCH_USERNAME,
 						password: process.env.ELASTICSEARCH_PASSWORD,
 					},
-					flushInterval: 10000,
+					flushInterval: 1000,
+					'flush-bytes': 1000,
 				}
 				if (elasticConfig) {
 					Object.assign(esConfig, elasticConfig)
 				}
+
 				const esTransport = pinoElastic(esConfig)
+
 				transports.push(esTransport)
 			} else {
 				transports.push(
 					pino.destination({
 						minLength: 1024,
-						sync: true,
+						sync: false,
 					})
 				)
 			}
+
 			pinoLogger = pino(
 				{
 					level: process.env.LOG_LEVEL,
@@ -95,10 +99,17 @@ class Logger {
 	}
 
 	private log(logLevel: LOG_LEVEL, logEvent: LogEvent, ...args: unknown[]) {
+		let detail
+		if (process.env.NODE_ENV === 'local' || process.env.NODE_ENV === 'test') {
+			detail = args
+		} else {
+			//@ts-ignore
+			detail = JSON.stringify(...args)
+		}
 		this._logger[logLevel]({
 			component: this._name,
 			...logEvent,
-			detail: args,
+			detail,
 		})
 	}