Merge pull request #2 from sitespeedio/catch-errors

Catch all logging errors
sitespeedio · Jan 10, 2025 · 8407431 · 8407431
2 parents 8a2a702 + 2923de7
commit 8407431
Showing 1 changed file with 22 additions and 17 deletions.
diff --git a/src/logger.js b/src/logger.js
@@ -180,28 +180,33 @@ export class Logger {
   }
 
   log(level, message, ...args) {
-    const currentLevel = loggerConfig.level.toLowerCase();
-    if (LOG_LEVELS[level] > LOG_LEVELS[currentLevel]) {
-      return;
-    }
+    try {
+      const currentLevel = loggerConfig.level.toLowerCase();
+      if (LOG_LEVELS[level] > LOG_LEVELS[currentLevel]) {
+        return;
+      }
 
-    const timestamp = getTimestamp();
-    const upperLevel = level.toUpperCase();
+      const timestamp = getTimestamp();
+      const upperLevel = level.toUpperCase();
 
-    let formattedMessage = formatPlaceholders(message, args);
+      let formattedMessage = formatPlaceholders(message, args);
 
-    const finalOutput = loggerConfig.format
-      .replace('%(date)s', timestamp)
-      .replace('%(levelname)s', upperLevel)
-      .replace('%(name)s', this.name)
-      .replace('%(message)s', formattedMessage);
+      const finalOutput = loggerConfig.format
+        .replace('%(date)s', timestamp)
+        .replace('%(levelname)s', upperLevel)
+        .replace('%(name)s', this.name)
+        .replace('%(message)s', formattedMessage);
 
-    const consoleMethod = CONSOLE_METHOD_MAP[level] ?? 'log';
-    const colorPrefix = COLOR_MAP[level] ?? '';
-    const colorSuffix = colorPrefix ? COLORS.reset : '';
-    const coloredOutput = `${colorPrefix}${finalOutput}${colorSuffix}`;
+      const consoleMethod = CONSOLE_METHOD_MAP[level] ?? 'log';
+      const colorPrefix = COLOR_MAP[level] ?? '';
+      const colorSuffix = colorPrefix ? COLORS.reset : '';
+      const coloredOutput = `${colorPrefix}${finalOutput}${colorSuffix}`;
 
-    console[consoleMethod](coloredOutput);
+      console[consoleMethod](coloredOutput);
+    } catch (error) {
+      console.error('Could create the log message');
+      console.error(`${message} with the error ${error}`);
+    }
   }
 
   critical(message, ...args) {