Fix logger so that error stacks are also logged on console

src/config/logger.js

@@ -1,17 +1,24 @@
 const winston = require('winston');
 const config = require('./config');
 
+const enumerateErrorFormat = winston.format(info => {
+  if (info instanceof Error) {
+    Object.assign(info, { message: info.stack });
+  }
+  return info;
+});
+
 const logger = winston.createLogger({
   level: config.env === 'development' ? 'debug' : 'info',
+  format: winston.format.combine(
+    enumerateErrorFormat(),
+    config.env === 'development' ? winston.format.colorize() : winston.format.uncolorize(),
+    winston.format.splat(),
+    winston.format.printf(({ level, message }) => `${level}: ${message}`)
+  ),
   transports: [
     new winston.transports.Console({
       stderrLevels: ['error'],
-      format: winston.format.combine(
-        config.env === 'development' ? winston.format.colorize() : winston.format.uncolorize(),
-        winston.format.splat(),
-        winston.format.printf(({ level, message }) => `${level}: ${message}`)
-      ),
-      handleExceptions: true,
     }),
   ],
 });

src/config/mongoose.js

@@ -1,11 +1,5 @@
 const mongoose = require('mongoose');
 const config = require('./config');
-const logger = require('./logger');
-
-mongoose.connection.on('error', error => {
-  logger.error(`MongoDB connection error ${error}`);
-  process.exit(1);
-});
 
 const connect = async () => {
   return mongoose.connect(config.mongodbUrl, {

src/index.js

@@ -21,7 +21,7 @@ const exitHandler = () => {
 };
 
 const unexpectedErrorHandler = error => {
-  logger.error(`Unexpected error ${error}`);
+  logger.error(error);
   exitHandler();
 };
 

src/middlewares/error.js

@@ -1,5 +1,6 @@
 const httpStatus = require('http-status');
 const config = require('../config/config');
+const logger = require('../config/logger');
 const AppError = require('../utils/AppError');
 
 const errorConverter = (err, req, res, next) => {
@@ -28,6 +29,10 @@ const errorHandler = (err, req, res, next) => {
     ...(config.env === 'development' && { stack: err.stack }),
   };
 
+  if (config.env === 'development') {
+    logger.error(err);
+  }
+
   res.status(statusCode).send(response);
 };