From 4ffe9438e0db8172c4223da27d9007d17c1a0b90 Mon Sep 17 00:00:00 2001
From: pierreozoux <pierre@ozoux.net>
Date: Wed, 22 Nov 2017 16:10:03 +0100
Subject: [PATCH] Fix add logger bug

In case you want to have only the console, it will fail at the `Winston.add`
as it is already there.
---
 server/logger.js | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/server/logger.js b/server/logger.js
index 70caa2929..ae4a6338c 100644
--- a/server/logger.js
+++ b/server/logger.js
@@ -3,6 +3,8 @@ Meteor.startup(() => {
   require('winston-zulip');
   const fs = require('fs');
 
+  //remove default logger
+  Winston.remove(Winston.transports.Console);
   const loggerEnable = process.env.LOGGER_ENABLE || false;
   if (loggerEnable) {
 
@@ -15,9 +17,6 @@ Meteor.startup(() => {
         json: true,
         timestamp: true,
       });
-    } else {
-      //remove default logger
-      Winston.remove(Winston.transports.Console);
     }
 
     if (loggers.includes('file')) {
@@ -57,9 +56,6 @@ Meteor.startup(() => {
       Winston.log('info', `zulipconfig ${zulipConfig}`);
     }
 
-  } else {
-    //remove default logger
-    Winston.remove(Winston.transports.Console);
   }
   Winston.log('info', 'Logger is completly instanciate');
 });