ml-testing-toolkit 18.15.1 → 18.16.0

package/.ncurc.yaml

@@ -4,6 +4,7 @@ reject: [
   "postman-collection",
   "postman-sandbox",
   "jest",
-  "@types/jest"
+  "@types/jest",
+  "uuid",
 ]
 

package/CHANGELOG.md

@@ -1,4 +1,11 @@
 # Changelog: [mojaloop/thirdparty-api-svc](https://github.com/mojaloop/thirdparty-api-svc)
+## [18.16.0](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.15.1...v18.16.0) (2025-09-16)
+
+
+### Bug Fixes
+
+* **csi-1782:** fix MyEmitter memory leak ([#342](https://github.com/mojaloop/ml-testing-toolkit/issues/342)) ([c141048](https://github.com/mojaloop/ml-testing-toolkit/commit/c1410487a3c17824b0869b6920c2132d491a6007))
+
 ### [18.15.1](https://github.com/mojaloop/ml-testing-toolkit/compare/v18.15.0...v18.15.1) (2025-09-01)
 
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ml-testing-toolkit",
   "description": "Testing Toolkit for Mojaloop implementations",
-  "version": "18.15.1",
+  "version": "18.16.0",
   "license": "Apache-2.0",
   "author": "Vijaya Kumar Guthi, ModusBox Inc. ",
   "contributors": [
@@ -78,23 +78,23 @@
     "@hapi/hapi": "21.4.3",
     "@hapi/inert": "7.1.0",
     "@hapi/vision": "7.0.3",
-    "@mojaloop/central-services-logger": "11.9.1",
-    "@mojaloop/central-services-metrics": "12.6.0",
-    "@mojaloop/ml-schema-transformer-lib": "2.7.7",
+    "@mojaloop/central-services-logger": "11.9.3",
+    "@mojaloop/central-services-metrics": "12.7.1",
+    "@mojaloop/ml-schema-transformer-lib": "2.7.8",
     "@mojaloop/ml-testing-toolkit-shared-lib": "14.2.0",
-    "@mojaloop/sdk-standard-components": "19.16.7",
+    "@mojaloop/sdk-standard-components": "19.17.0",
     "@now-ims/hapi-now-auth": "2.1.0",
     "@types/socket.io": "3.0.2",
     "adm-zip": "0.5.16",
     "ajv-formats": "3.0.1",
     "atob": "2.1.2",
-    "axios": "1.11.0",
+    "axios": "1.12.2",
     "chai": "4.4.1",
     "connection-string": "^5.0.0",
     "cookie-parser": "1.4.7",
     "cookies": "0.9.1",
     "cors": "2.8.5",
-    "dotenv": "17.2.1",
+    "dotenv": "17.2.2",
     "express": "5.1.0",
     "express-validator": "7.2.1",
     "fs": "0.0.1-security",
@@ -108,7 +108,7 @@
     "json-rules-engine": "7.3.1",
     "jsonwebtoken": "9.0.2",
     "lodash": "4.17.21",
-    "mongoose": "8.18.0",
+    "mongoose": "8.18.1",
     "multer": "2.0.2",
     "mustache": "4.2.0",
     "mv": "2.1.1",
@@ -138,7 +138,7 @@
     "jest": "29.7.0",
     "jest-junit": "16.0.0",
     "nodemon": "3.1.10",
-    "npm-check-updates": "18.0.3",
+    "npm-check-updates": "18.1.1",
     "nyc": "17.1.0",
     "parse-strings-in-object": "1.6.0",
     "pre-commit": "1.2.2",

package/src/lib/eventListenerClient/inboundEventListener.js

@@ -47,6 +47,12 @@ class InboundEventListener {
 
   async init () {
     this.userConfig = await Config.getStoredUserConfig()
+
+    // Fixes the MyEmitter listener leak issue
+    if (this.emitter.listenerCount('newInbound') > 0) {
+      this.emitter.removeAllListeners('newInbound')
+    }
+
     this.emitter.on('newInbound', (data) => {
       for (const [, eventListener] of Object.entries(this.eventListeners)) {
         // Match method, path and condition for each inbound request
@@ -98,59 +104,74 @@ class InboundEventListener {
 
   getMessage (clientName, timeout = 5000) {
     return new Promise((resolve, reject) => {
+      let timer = null
+
+      const cleanup = () => {
+        if (timer) {
+          clearTimeout(timer)
+          timer = null
+        }
+        if (this.eventListeners[clientName] && this.eventListeners[clientName].eventEmitter) {
+          this.eventListeners[clientName].eventEmitter.removeAllListeners('newMessage')
+        }
+      }
+
       if (!this.eventListeners[clientName]) {
         resolve(null)
-      } else {
-        // Check for the message received already
-        if (this.eventListeners[clientName].message !== null) {
-          // Store the message somewhere
-          const retMessage = this.parseMessage(this.eventListeners[clientName].message)
+        return
+      }
+
+      // Check for the message received already
+      if (this.eventListeners[clientName].message !== null) {
+        // Store the message somewhere
+        const retMessage = this.parseMessage(this.eventListeners[clientName].message)
+        // Destroy the event listener
+        this.destroy(clientName)
+        // Return the stored message
+        // this.customLog('Returning stored message...')
+        if (this.transformer.reverseTransform) {
+          this.transformer.reverseTransform(retMessage).then((result) => {
+            resolve(result)
+          }).catch((err) => {
+            this.customLog('Error transforming message: ' + err)
+            resolve(retMessage)
+          })
+        } else {
+          resolve(retMessage)
+        }
+        return
+      }
+
+      // Listen for the new message for some time
+      // Set the timer
+      timer = setTimeout(() => {
+        try {
+          this.customLog('Error: Timeout')
           // Destroy the event listener
           this.destroy(clientName)
-          // Return the stored message
-          // this.customLog('Returning stored message...')
-          if (this.transformer.reverseTransform) {
-            this.transformer.reverseTransform(retMessage).then((result) => {
-              resolve(result)
-            }).catch((err) => {
-              this.customLog('Error transforming message: ' + err)
-              resolve(retMessage)
-            })
-          } else {
+          resolve(null)
+        } catch (err) {
+          reject(err)
+        }
+      }, timeout)
+
+      // Listen for message
+      this.eventListeners[clientName].eventEmitter.once('newMessage', (message) => {
+        cleanup()
+        this.destroy(clientName)
+        const retMessage = this.parseMessage(message)
+
+        if (this.transformer.reverseTransform) {
+          this.transformer.reverseTransform(retMessage).then((result) => {
+            resolve(result)
+          }).catch((err) => {
+            this.customLog('Error transforming message: ' + err)
             resolve(retMessage)
-          }
-        } else {
-          // Listen for the new message for some time
-          let timer = null
-          // Set the timer
-          timer = setTimeout(() => {
-            try {
-              this.customLog('Error: Timeout')
-              // Destroy the event listener
-              this.destroy(clientName)
-              resolve(null)
-            } catch (err) {
-              reject(err)
-            }
-          }, timeout)
-          // Listen for message
-          this.eventListeners[clientName].eventEmitter.once('newMessage', (message) => {
-            clearTimeout(timer)
-            this.destroy(clientName)
-            const retMessage = this.parseMessage(message)
-            if (this.transformer.reverseTransform) {
-              this.transformer.reverseTransform(retMessage).then((result) => {
-                resolve(result)
-              }).catch((err) => {
-                this.customLog('Error transforming message: ' + err)
-                resolve(retMessage)
-              })
-            } else {
-              resolve(retMessage)
-            }
           })
+        } else {
+          resolve(retMessage)
         }
-      }
+      })
     })
   }
 

package/src/lib/webSocketClient/WebSocketClientManager.js

@@ -53,6 +53,8 @@ class WebSocketClientManager {
   }
 
   connect (url, clientName, timeout = 15000) {
+    this.customLog(`Connecting to WebSocket URL: ${url} with client name: ${clientName} and timeout: ${timeout}ms`)
+
     const tlsOptions = { rejectUnauthorized: false }
     const urlObject = new URL(url)
     if (this.userConfig.CLIENT_MUTUAL_TLS_ENABLED) {
@@ -107,41 +109,48 @@ class WebSocketClientManager {
 
   getMessage (clientName, timeout = 5000) {
     return new Promise((resolve, reject) => {
+      let timer = null
+
+      const cleanup = () => {
+        if (timer) {
+          clearTimeout(timer)
+          timer = null
+        }
+        if (this.ws[clientName] && this.ws[clientName].eventEmitter) {
+          this.ws[clientName].eventEmitter.removeAllListeners('newMessage')
+        }
+      }
+
       if (!this.ws[clientName]) {
         resolve(null)
-      } else {
-        // Check for the message received already
-        if (this.ws[clientName].message !== null) {
-          // Store the message somewhere
-          const retMessage = this.parseMessage(this.ws[clientName].message)
-          // Disconnect the websocket connection
+        return
+      }
+
+      // Check for already received message
+      if (this.ws[clientName].message !== null) {
+        const retMessage = this.parseMessage(this.ws[clientName].message)
+        this.disconnect(clientName)
+        resolve(retMessage)
+        return
+      }
+
+      // Set timeout
+      timer = setTimeout(() => {
+        try {
+          cleanup()
           this.disconnect(clientName)
-          // Return the stored message
-          // this.customLog('Returning stored message...')
-          resolve(retMessage)
-        } else {
-          // Listen for the new message for some time
-          let timer = null
-          // Set the timer
-          timer = setTimeout(() => {
-            try {
-              this.ws[clientName].eventEmitter.removeAllListeners('newMessage')
-              // Disconnect the websocket connection
-              this.disconnect(clientName)
-              resolve(null)
-            } catch (err) {
-              reject(err)
-            }
-          }, timeout)
-          // Listen for message
-          this.ws[clientName].eventEmitter.once('newMessage', (message) => {
-            clearTimeout(timer)
-            this.ws[clientName].eventEmitter.removeAllListeners('newMessage')
-            this.disconnect(clientName)
-            resolve(this.parseMessage(message))
-          })
+          resolve(null)
+        } catch (err) {
+          reject(err)
         }
-      }
+      }, timeout)
+
+      // Listen for message
+      this.ws[clientName].eventEmitter.once('newMessage', (message) => {
+        cleanup()
+        this.disconnect(clientName)
+        resolve(this.parseMessage(message))
+      })
     })
   }
 
@@ -155,7 +164,10 @@ class WebSocketClientManager {
 
   disconnect (clientName) {
     if (this.ws[clientName]) {
-      this.ws[clientName].client.close()
+      if (this.ws[clientName].client) {
+        this.ws[clientName].client.removeAllListeners()
+        this.ws[clientName].client.close()
+      }
       if (this.ws[clientName].eventEmitter) {
         this.ws[clientName].eventEmitter.removeAllListeners()
       }
@@ -170,7 +182,10 @@ class WebSocketClientManager {
 
   disconnectAll () {
     for (const clientName in this.ws) {
-      this.ws[clientName].client.close()
+      if (this.ws[clientName].client) {
+        this.ws[clientName].client.removeAllListeners()
+        this.ws[clientName].client.close()
+      }
       if (this.ws[clientName].eventEmitter) {
         this.ws[clientName].eventEmitter.removeAllListeners()
       }
@@ -183,13 +198,6 @@ class WebSocketClientManager {
     }
     return true
   }
-
-  clearAllTimers () {
-    Object.keys(this.timers).forEach((clientName) => {
-      clearTimeout(this.timers[clientName])
-      delete this.timers[clientName]
-    })
-  }
 }
 
 module.exports = {