fix: improvemnts to logger boot process and messages

wmfs · Dec 1, 2021 · bce4cdc · bce4cdc
1 parent f3c6eea
commit bce4cdc
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/lib/plugin/components/services/logger/index.js b/lib/plugin/components/services/logger/index.js
@@ -10,11 +10,19 @@ const { isPlainObject } = require('lodash')
 class Logger {
   boot (options) {
     this.shouldLog = [...LEVELS, 'ON', '*'].includes(process.env.LOGGER)
-    this.level = LEVELS.includes(process.env.LOGGER)
-      ? process.env.LOGGER
-      : (process.env.LOGGER === '*' ? 'trace' : 'info')
-    this.loggerOutputFilePath = generateFilePath(process.env.LOGGER_OUTPUT_DIR_PATH)
-    options.messages.info(this.loggerOutputFilePath ? `Output logs to ${this.loggerOutputFilePath}` : 'No output file')
+
+    if (this.shouldLog) {
+      this.level = LEVELS.includes(process.env.LOGGER)
+        ? process.env.LOGGER
+        : (process.env.LOGGER === '*' ? 'trace' : 'info')
+      this.loggerOutputFilePath = generateFilePath(process.env.LOGGER_OUTPUT_DIR_PATH)
+
+      options.messages.info(`Logger level set to: ${this.level}`)
+      options.messages.info(this.loggerOutputFilePath ? `Output logs to ${this.loggerOutputFilePath}` : 'No output file')
+    } else {
+      options.messages.info('Using default logger')
+    }
+
     this.logger = this.createLogger()
 
     const createLogFn = level => { this[level] = msg => this.logger[level](msg) }