Merge pull request #199 from AthennaIO/develop

fix(ignite): register only one uncaught ehandler
AthennaIO · Dec 10, 2023 · 0ae1b64 · 0ae1b64
2 parents 1f4cd39 + ef0ab01
commit 0ae1b64
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@athenna/core",
-  "version": "4.14.0",
+  "version": "4.14.1",
   "description": "The plug and play Node.js framework.",
   "license": "MIT",
   "author": "João Lenon <[email protected]>",

diff --git a/src/ignite/Ignite.ts b/src/ignite/Ignite.ts
@@ -193,11 +193,14 @@ export class Ignite {
    * ```
    */
   public setUncaughtExceptionHandler(): void {
+    /**
+     * Remove listeners registered more then once by
+     * Ignite class.
+     */
     if (process.listeners('uncaughtException').length) {
-      process.removeListener(
-        'uncaughtException',
-        this.options.uncaughtExceptionHandler
-      )
+      process.listeners('uncaughtException').forEach(l => {
+        process.removeListener('uncaughtException', l)
+      })
     }
 
     process.on('uncaughtException', this.options.uncaughtExceptionHandler)

diff --git a/tests/unit/ignite/IgniteTest.ts b/tests/unit/ignite/IgniteTest.ts
@@ -132,7 +132,7 @@ export default class IgniteTest {
 
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-ignore
-    assert.lengthOf(process?._events?.uncaughtException, 3)
+    assert.lengthOf(process?._events?.uncaughtException, 2)
   }
 
   @Test()