Merge pull request #137 from Sebastian-Webster/136-zombie-processes-are-created-when-exiting-nodejs-with-ctrl-+-c

Sebastian-Webster · web-flow · commit 5a2783c06fbc · 2024-11-15T15:52:07.000+13:00
Fix zombie processes
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -40,7 +40,8 @@
   },
   "dependencies": {
     "adm-zip": "^0.5.16",
-    "semver": "^7.6.3"
+    "semver": "^7.6.3",
+    "signal-exit": "^4.1.0"
   },
   "repository": {
     "type": "git",
diff --git a/src/libraries/AbortSignal.ts b/src/libraries/AbortSignal.ts
@@ -1,3 +1,5 @@
+import { onExit } from 'signal-exit'
+
 const DBDestroySignal = new AbortController();
 
 function abortSignal() {
@@ -6,7 +8,6 @@ function abortSignal() {
     }
 }
 
-process.on('beforeExit', abortSignal)
-process.on('exit', abortSignal)
+onExit(abortSignal)
 
 export default DBDestroySignal;

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+import { onExit } from 'signal-exit'`
	`2`	`+`
`1`	`3`	`const DBDestroySignal = new AbortController();`
`2`	`4`
`3`	`5`	`function abortSignal() {`
`@@ -6,7 +8,6 @@ function abortSignal() {`
`6`	`8`	`}`
`7`	`9`	`}`
`8`	`10`
`9`		`-process.on('beforeExit', abortSignal)`
`10`		`-process.on('exit', abortSignal)`
	`11`	`+onExit(abortSignal)`
`11`	`12`
`12`	`13`	`export default DBDestroySignal;`