From cb52dd4fa5478281a46e9b73cec9e0bc36692103 Mon Sep 17 00:00:00 2001
From: Apipol Niyomsak <makemek101@hotmail.com>
Date: Sun, 30 Apr 2017 00:17:16 +0700
Subject: [PATCH] config: remove LOGGER_ENABLE option

LOGGER_LEVEL has 'silent' which disable a log.
Having LOGGER_ENABLE is redundant.

PR-URL: https://github.com/makemek/Modcolle/pull/46
---
 README.md     | 1 -
 process.yml   | 1 -
 src/logger.js | 1 -
 3 files changed, 3 deletions(-)

diff --git a/README.md b/README.md
index b30f18d..966d0fd 100644
--- a/README.md
+++ b/README.md
@@ -72,7 +72,6 @@ PORT_DEV=<port number> # browser-sync listeing port; default is 3000
 At `env` section in `process.yml` are environment variables that will override `.env.json` for running in production environment.
 
 ### Environment Variables
-- LOGGER_ENABLE: if `true` enables log, `false` disables log
 - LOGGER_LEVEL: select a [logging level](https://github.com/pinojs/pino/blob/master/docs/API.md#level)
 - LOGGER_PRETTY: pretty print log (should be set to `false` in production)
 - PORT: application port
diff --git a/process.yml b/process.yml
index ea110c5..582ca30 100644
--- a/process.yml
+++ b/process.yml
@@ -10,7 +10,6 @@ apps:
      pid_file: ./log/modcolle.pid
 
      env:
-      LOGGER_ENABLE: true
       LOGGER_PRETTY: true
       LOGGER_LEVEL: info
       PORT: 5000
diff --git a/src/logger.js b/src/logger.js
index 16cba40..1922514 100644
--- a/src/logger.js
+++ b/src/logger.js
@@ -11,7 +11,6 @@ const loggers = {}
 labels.forEach(label => {
   loggers[label] = pino({
     name: label,
-    enabled: LOGGER_ENABLE === 'true',
     level: LOGGER_LEVEL,
     prettyPrint: LOGGER_PRETTY === 'true'
   })