Merge pull request #310 from clark800/fix/logger

Make api logger print stack trace even in NODE_ENV=test
ripple · Mar 10, 2015 · 8101bae · 8101bae
2 parents 4813aad + 2a33712
commit 8101bae
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/api/lib/logger.js b/api/lib/logger.js
@@ -7,13 +7,20 @@ var logger = new winston.Logger({
       prettyPrint: true,
       colorize: true,
       timestamp: true,
-      handleExceptions: true,
+      handleExceptions: false,
       logLevel: 'info',
       showLevel: true,
       silent: config.get('NODE_ENV') === 'test'
     })
   ],
-  exitOnError: false
+  exceptionHandlers: [
+    new winston.transports.Console({
+      prettyPrint: true,
+      colorize: true,
+      timestamp: true
+    })
+  ],
+  exitOnError: true
 });
 
 exports.logger = logger;
diff --git a/server/logger.js b/server/logger.js
@@ -9,14 +9,21 @@ var logger = exports.logger = new winston.Logger({
     new winston.transports.Console({
       prettyPrint: true,
       colorize: true,
-      handleExceptions: true,
+      handleExceptions: false,
       level: 'info',
       showLevel: true,
       timestamp: true,
       silent: config.get('NODE_ENV') === 'test'
     })
   ],
-  exitOnError: false
+  exceptionHandlers: [
+    new winston.transports.Console({
+      prettyPrint: true,
+      colorize: true,
+      timestamp: true
+    })
+  ],
+  exitOnError: true
 });
 
 /**