Merge pull request #254 from lirantal/enhancement-logger-config

Enhancing application logger configuration

config/env/all.js

@@ -10,6 +10,15 @@ module.exports = {
 	templateEngine: 'swig',
 	sessionSecret: 'MEAN',
 	sessionCollection: 'sessions',
+	log: {
+		// Can specify one of 'combined', 'common', 'dev', 'short', 'tiny'
+		format: 'combined',
+		// Stream defaults to process.stdout
+		// Uncomment to enable logging to a log on the file system
+		options: {
+			stream: 'access.log'
+		}
+	},
 	assets: {
 		lib: {
 			css: [

config/env/development.js

@@ -2,6 +2,15 @@
 
 module.exports = {
 	db: 'mongodb://localhost/mean-dev',
+	log: {
+		// Can specify one of 'combined', 'common', 'dev', 'short', 'tiny'
+		format: 'dev',
+		// Stream defaults to process.stdout
+		// Uncomment to enable logging to a log on the file system
+		options: {
+			//stream: 'access.log'
+		}
+	},
 	app: {
 		title: 'MEAN.JS - Development Environment'
 	},

config/env/production.js

@@ -2,6 +2,15 @@
 
 module.exports = {
 	db: process.env.MONGOHQ_URL || process.env.MONGOLAB_URI || 'mongodb://' + (process.env.DB_1_PORT_27017_TCP_ADDR || 'localhost') + '/mean',
+	log: {
+		// Can specify one of 'combined', 'common', 'dev', 'short', 'tiny'
+		format: 'combined',
+		// Stream defaults to process.stdout
+		// Uncomment to enable logging to a log on the file system
+		options: {
+			stream: 'access.log'
+		}
+	},
 	assets: {
 		lib: {
 			css: [

config/env/secure.js

@@ -3,6 +3,15 @@
 module.exports = {
 	port: 443,
 	db: process.env.MONGOHQ_URL || process.env.MONGOLAB_URI || 'mongodb://localhost/mean',
+	log: {
+		// Can specify one of 'combined', 'common', 'dev', 'short', 'tiny'
+		format: 'combined',
+		// Stream defaults to process.stdout
+		// Uncomment to enable logging to a log on the file system
+		options: {
+			stream: 'access.log'
+		}
+	},
 	assets: {
 		lib: {
 			css: [

config/env/test.js

@@ -3,6 +3,15 @@
 module.exports = {
 	db: 'mongodb://localhost/mean-test',
 	port: 3001,
+	log: {
+		// Can specify one of 'combined', 'common', 'dev', 'short', 'tiny'
+		format: 'dev',
+		// Stream defaults to process.stdout
+		// Uncomment to enable logging to a log on the file system
+		options: {
+			//stream: 'access.log'
+		}
+	},
 	app: {
 		title: 'MEAN.JS - Test Environment'
 	},

config/express.js

@@ -8,6 +8,7 @@ var fs = require('fs'),
 	https = require('https'),
 	express = require('express'),
 	morgan = require('morgan'),
+	logger = require('./logger'),
 	bodyParser = require('body-parser'),
 	session = require('express-session'),
 	compress = require('compression'),
@@ -64,11 +65,11 @@ module.exports = function(db) {
 	app.set('view engine', 'server.view.html');
 	app.set('views', './app/views');
 
+	// Enable logger (morgan)
+	app.use(morgan(logger.getLogFormat(), logger.getLogOptions()));
+
 	// Environment dependent middleware
 	if (process.env.NODE_ENV === 'development') {
-		// Enable logger (morgan)
-		app.use(morgan('dev'));
-
 		// Disable views cache
 		app.set('view cache', false);
 	} else if (process.env.NODE_ENV === 'production') {

config/logger.js

@@ -0,0 +1,36 @@
+'use strict';
+
+/**
+ * Module dependencies.
+ */
+
+var morgan = require('morgan');
+var config = require('./config');
+var fs = require('fs');
+
+/**
+ * Module init function.
+ */
+module.exports = {
+
+	getLogFormat: function() {
+		return config.log.format;
+	},
+
+	getLogOptions: function() {
+		var options = {};
+
+		try {
+			if ('stream' in config.log.options) {
+				options = {
+					stream: fs.createWriteStream(process.cwd() + '/' + config.log.options.stream, {flags: 'a'})
+				};
+			}
+		} catch (e) {
+			options = {};
+		}
+
+		return options;
+	}
+
+};