Fix: do not colorize non-string log message

Kreeldor · Oct 14, 2023 · a362206 · a362206
1 parent f6bdaac
commit a362206
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 7 deletions.
diff --git a/server/monitor-types/real-browser-monitor-type.js b/server/monitor-types/real-browser-monitor-type.js
@@ -49,8 +49,6 @@ if (process.platform === "win32") {
     ];
 }
 
-log.debug("chrome", allowedList);
-
 /**
  * Is the executable path allowed?
  * @param {string} executablePath Path to executable

diff --git a/src/util.js b/src/util.js
@@ -147,13 +147,25 @@ class Logger {
             console.warn(timePart, modulePart, levelPart, msg);
         }
         else if (level === "ERROR") {
-            let msgPart = exports.CONSOLE_STYLE_FgRed + msg + exports.CONSOLE_STYLE_Reset;
+            let msgPart;
+            if (typeof msg === "string") {
+                msgPart = exports.CONSOLE_STYLE_FgRed + msg + exports.CONSOLE_STYLE_Reset;
+            }
+            else {
+                msgPart = msg;
+            }
             console.error(timePart, modulePart, levelPart, msgPart);
         }
         else if (level === "DEBUG") {
             if (exports.isDev) {
                 timePart = exports.CONSOLE_STYLE_FgGray + now + exports.CONSOLE_STYLE_Reset;
-                let msgPart = exports.CONSOLE_STYLE_FgGray + msg + exports.CONSOLE_STYLE_Reset;
+                let msgPart;
+                if (typeof msg === "string") {
+                    msgPart = exports.CONSOLE_STYLE_FgGray + msg + exports.CONSOLE_STYLE_Reset;
+                }
+                else {
+                    msgPart = msg;
+                }
                 console.debug(timePart, modulePart, levelPart, msgPart);
             }
         }

diff --git a/src/util.ts b/src/util.ts
@@ -208,13 +208,23 @@ class Logger {
         } else if (level === "WARN") {
             console.warn(timePart, modulePart, levelPart, msg);
         } else if (level === "ERROR") {
-            let msgPart = CONSOLE_STYLE_FgRed + msg + CONSOLE_STYLE_Reset;
+            let msgPart :string;
+            if (typeof msg === "string") {
+                msgPart = CONSOLE_STYLE_FgRed + msg + CONSOLE_STYLE_Reset;
+            } else {
+                msgPart = msg;
+            }
             console.error(timePart, modulePart, levelPart, msgPart);
         } else if (level === "DEBUG") {
             if (isDev) {
                 timePart = CONSOLE_STYLE_FgGray + now + CONSOLE_STYLE_Reset;
-                let msgPart = CONSOLE_STYLE_FgGray + msg + CONSOLE_STYLE_Reset;
-                console.debug(timePart, modulePart, levelPart, msgPart );
+                let msgPart :string;
+                if (typeof msg === "string") {
+                    msgPart = CONSOLE_STYLE_FgGray + msg + CONSOLE_STYLE_Reset;
+                } else {
+                    msgPart = msg;
+                }
+                console.debug(timePart, modulePart, levelPart, msgPart);
             }
         } else {
             console.log(timePart, modulePart, msg);