feat: Merge pull request #42 from pelias/avoid-process.exit

Explicitly close HTTP server to avoid process.exit
pelias · Dec 1, 2017 · 766da53 · 766da53
2 parents 4d77c7d + f1f2cf2
commit 766da53
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -1,5 +1,6 @@
+const app = require('./app')();
+
 try {
-  const app = require('./app')();
   const port = ( parseInt(process.env.PORT) || 3102 );
 
   app.listen(port, () => {
@@ -8,6 +9,8 @@ try {
 
 } catch (err) {
   console.error(err);
-  process.exit(1);
 
+  // use exitCode to exit safely: https://nodejs.org/api/process.html#process_process_exit_code
+  process.exitCode = 1;
+  app.close();
 }