added real close sequence and a bit more logging

index.js

@@ -19,19 +19,22 @@ shared.config=function(namespace, key) {
 	return config[namespace][key];
 };
 
+// create global event emitter
+const EE=require('events');
+const events=new EE();
+shared.events=events;
+
 // load database
 require('./src/db');
 
 // load everything
 require('./src/express');
 
 // setup signal handlers to exit
-((a => {
-	[].forEach.call(['int', 'term', 'hup'], a);
-})(s => {
+['int', 'term', 'hup'].forEach(s => {
 	s='SIG'+s.toUpperCase();
 	process.on(s, () => {
-		log.notice("Exiting on "+s);
-		process.exit(0);
+		log.notice("Exiting due to "+s);
+		events.emit('die', s);
 	});
-}));
+});

src/db.js

@@ -24,10 +24,10 @@ try {
 }
 
 // add close handlers
-process.on('exit', () => db.close());
-process.on('SIGINT', () => db.close());
-process.on('SIGHUP', () => db.close());
-process.on('SIGTERM', () => db.close());
+shared.events.on('die', () => {
+	db.close();
+	log.notice("Closed database");
+});
 
 // execute db migrations automatically
 (() => {
@@ -57,7 +57,7 @@ process.on('SIGTERM', () => db.close());
 	}
 });
 
-log.info("Database is ready");
+log.notice("Database is ready");
 
 shared.db=db;
 module.exports=db;

src/express.js

@@ -1,3 +1,8 @@
+// load libs
+const shared=require('../shared');
+const db=require('./db');
+const log=require('log').get('express');
+
 // create and setup an app
 const express=require('express');
 const app=express();
@@ -9,5 +14,12 @@ app.set('views', process.cwd()+'/views'); // allow EJS views to be loaded
 app.use(express.static(process.cwd()+'/static')); // allow static resources to be served
 
 // listen on either the port given in env (or loaded from config) or 3000
-app.listen(process.env.PORT || 3000, process.env.ADDRESS || '0.0.0.0');
+const server=app.listen(process.env.PORT || 3000, process.env.ADDRESS || '0.0.0.0', () => {
+	log.notice("Webserver listening on "+(process.env.ADDRESS || '0.0.0.0')+":"+(process.env.PORT || 3000));
+});
 
+// setup exit handler
+shared.events.on('die', () => {
+	log.notice("Closing webserver");
+	server.close(() => log.notice("Closed webserver"));
+});