[pinpoint-apm#117] logger loading time

feelform · Oct 20, 2022 · 1a8d8fa · 1a8d8fa
1 parent dc863d6
commit 1a8d8fa
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/lib/supports.js b/lib/supports.js
@@ -9,7 +9,7 @@
 const Logger = require('./utils/log/logger2')
 const { getConfig, needsLoadConfig } = require('./config')
 
-const log = new Logger.NoneBuilder({
+const noneConfigurationLog = new Logger.NoneBuilder({
    logs: [],
    debug: function (message) {
       this.logs.push(message)
@@ -25,15 +25,21 @@ const log = new Logger.NoneBuilder({
    }
 }).build()
 
+let log = undefined
 module.exports = {
    getLog: function () {
-      if (needsLoadConfig()) {
+      if (log) {
          return log
       }
 
-      const config = getConfig()
-      if (config.logLevel) {
+      if (needsLoadConfig()) {
+         return noneConfigurationLog
+      }
 
+      const config = getConfig()
+      if (!log) {
+         log = Logger.makeBuilder(config.logLevel, {}).build()
       }
+      return log
    }
 }
diff --git a/test/utils/log/logger.integration.test.js b/test/utils/log/logger.integration.test.js
@@ -15,5 +15,6 @@ test('Logger Adaptor', (t) => {
     const actual = getLog()
     new Agent()
     t.equal(actual.adaptor.constructor.name, 'LoggerOutputAdaptor', 'log adaptor')
+    t.equal(actual.type.name, 'NONE', 'when first loading time, log type is NONE')
     t.end()
 })