@mhmdhammoud/meritt-utils 1.4.1 → 1.5.0

package/.github/workflows/npm-publish.yml

@@ -39,5 +39,5 @@ jobs:
           password: ${{ secrets.EMAIL_PASSWORD }}
           subject: ${{ github.event.head_commit.message }} ${{ github.job }} job of ${{ github.repository }} has ${{ job.status }}
           body: ${{ github.job }} job in worflow ${{ github.workflow }} of ${{ github.repository }} has ${{ job.status }}
-          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com,sawwas.omar@gmail.com
+          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com
           from: Github Action

package/.github/workflows/push.yml

@@ -11,7 +11,7 @@ jobs:
         with:
           node-version: 16
       - run: npm ci
-      - run: npm run types
+      - run: npm run test:types
   notify:
     needs: [type-check]
     runs-on: ubuntu-latest
@@ -25,5 +25,5 @@ jobs:
           password: ${{ secrets.EMAIL_PASSWORD }}
           subject: ${{ github.event.head_commit.message }} ${{ github.job }} job of ${{ github.repository }} has ${{ job.status }}
           body: ${{ github.job }} job in worflow ${{ github.workflow }} of ${{ github.repository }} has ${{ job.status }}
-          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com,sawwas.omar@gmail.com
+          to: mohammad.hammoud.lb@hotmail.com,steef12009@gmail.com
           from: Github Action

package/.husky/pre-commit

@@ -45,7 +45,7 @@ echo -ne '\n'
 
 echo -e "${yellow}Running types...${clear}"
 echo -ne '\n'
-npm run types
+npm run test:types
 echo -ne '\n'
 
 echo -e "${green}types passed...${clear} ✅"

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);
@@ -62,8 +63,8 @@ function getLogger(elasticConfig) {
             }
             else {
                 transports.push(pino_1.pino.destination({
-                    minLength: 1024,
-                    sync: true,
+                    minLength: 128,
+                    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.5.0",
 	"description": "",
 	"main": "./dist/index.js",
 	"private": false,
@@ -13,11 +13,11 @@
 		"access": "public"
 	},
 	"scripts": {
-		"dev-ts": "ts-node-dev --respawn --transpile-only src/index.ts  | pino-pretty",
+		"start:dev": "ts-node-dev --respawn --transpile-only src/index.ts  | pino-pretty",
 		"dev": "nodemon ./dist/index.js",
 		"watch": "tsc --watch",
 		"build": "tsc",
-		"types": "tsc --noemit",
+		"test:types": "tsc --noemit",
 		"prepare": "husky install",
 		"lint": "eslint src --fix",
 		"prepublish": "npm run build",

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,
+						minLength: 128,
+						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,
 		})
 	}