From 2ba326dfc6490d4d8e96ee44f6851c33f6c2c94a Mon Sep 17 00:00:00 2001
From: Evan Morikawa <evan@evanmorikawa.com>
Date: Fri, 2 Dec 2016 17:33:51 -0500
Subject: [PATCH] [local-sync] only enable logging in dev mode of N1

---
 packages/local-sync/src/local-sync-worker/sync-worker.js | 1 -
 packages/local-sync/src/shared/logger.es6                | 9 +++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/packages/local-sync/src/local-sync-worker/sync-worker.js b/packages/local-sync/src/local-sync-worker/sync-worker.js
index 04500ca6d..d80532fd8 100644
--- a/packages/local-sync/src/local-sync-worker/sync-worker.js
+++ b/packages/local-sync/src/local-sync-worker/sync-worker.js
@@ -176,7 +176,6 @@ class SyncWorker {
       return;
     }
 
-    console.log(this._account)
     this._logger.info({reason}, `SyncWorker: Account sync started`)
 
     try {
diff --git a/packages/local-sync/src/shared/logger.es6 b/packages/local-sync/src/shared/logger.es6
index f394a8c48..eae42aeb2 100644
--- a/packages/local-sync/src/shared/logger.es6
+++ b/packages/local-sync/src/shared/logger.es6
@@ -1,6 +1,12 @@
 const _ = require('underscore')
 
+let ENABLE_LOGGING = true;
+
 function Logger(boundArgs = {}) {
+  if (NylasEnv && !NylasEnv.inDevMode()) {
+    ENABLE_LOGGING = false
+  }
+
   if (!_.isObject(boundArgs)) {
     throw new Error('Logger: Bound arguments must be an object')
   }
@@ -8,6 +14,9 @@ function Logger(boundArgs = {}) {
   const loggerFns = ['log', 'info', 'warn', 'error']
   loggerFns.forEach((logFn) => {
     logger[logFn] = (first, ...args) => {
+      if (!ENABLE_LOGGING && logFn !== "error") {
+        return () => {}
+      }
       if (first instanceof Error || !_.isObject(first)) {
         if (_.isEmpty(boundArgs)) {
           return console[logFn](first, ...args)