From 3ae74557b484373cd20abcb236c047122ec35973 Mon Sep 17 00:00:00 2001
From: Stanislav A <s.atroschenko@adguard.com>
Date: Wed, 16 Nov 2022 14:16:21 +0300
Subject: [PATCH] fix logging in ga & log-on-stack-trace

---
 src/helpers/log-utils.js             |  6 +++---
 src/redirects/google-analytics-ga.js | 18 +++++++++++-------
 src/scriptlets/log-on-stack-trace.js |  6 ++++--
 3 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/src/helpers/log-utils.js b/src/helpers/log-utils.js
index 025acd3b..7062f6f1 100644
--- a/src/helpers/log-utils.js
+++ b/src/helpers/log-utils.js
@@ -1,11 +1,11 @@
 /**
  * Logs message to console.
  * Used when logging is a part of a scriptlet's functionality.
- * @param {string} message
+ * @param {...string} message
  */
-export const logMessage = (message) => {
+export const logMessage = (...args) => {
     // eslint-disable-next-line no-console
-    console.log(message);
+    console.log(...args);
 };
 
 /**
diff --git a/src/redirects/google-analytics-ga.js b/src/redirects/google-analytics-ga.js
index f271b23d..71d6c4e2 100644
--- a/src/redirects/google-analytics-ga.js
+++ b/src/redirects/google-analytics-ga.js
@@ -1,5 +1,9 @@
 /* eslint-disable no-underscore-dangle */
-import { hit, noopFunc } from '../helpers/index';
+import {
+    hit,
+    noopFunc,
+    logVerbose,
+} from '../helpers/index';
 
 /**
  * @redirect google-analytics-ga
@@ -93,11 +97,7 @@ export function GoogleAnalyticsGa(source) {
         try {
             window.location.assign(url);
         } catch (e) {
-            // log the error only while debugging
-            if (source.verbose) {
-                // eslint-disable-next-line no-console
-                console.log(e);
-            }
+            logVerbose(e, source.verbose);
         }
     };
 
@@ -128,4 +128,8 @@ GoogleAnalyticsGa.names = [
     'google-analytics_ga.js',
 ];
 
-GoogleAnalyticsGa.injections = [hit, noopFunc];
+GoogleAnalyticsGa.injections = [
+    hit,
+    noopFunc,
+    logVerbose,
+];
diff --git a/src/scriptlets/log-on-stack-trace.js b/src/scriptlets/log-on-stack-trace.js
index ee640a55..781a0be8 100644
--- a/src/scriptlets/log-on-stack-trace.js
+++ b/src/scriptlets/log-on-stack-trace.js
@@ -2,6 +2,7 @@ import {
     getPropertyInChain,
     setPropertyAccess,
     hit,
+    logMessage,
     // following helpers should be imported and injected
     // because they are used by helpers above
     isEmptyObject,
@@ -83,13 +84,13 @@ export function logOnStacktrace(source, property) {
         setPropertyAccess(base, prop, {
             get() {
                 hit(source);
-                console.log(`%cGet %c${prop}`, 'color:red;', 'color:green;');
+                logMessage(`%cGet %c${prop}`, 'color:red;', 'color:green;');
                 console.table(refineStackTrace(new Error().stack));
                 return value;
             },
             set(newValue) {
                 hit(source);
-                console.log(`%cSet %c${prop}`, 'color:red;', 'color:green;');
+                logMessage(`%cSet %c${prop}`, 'color:red;', 'color:green;');
                 console.table(refineStackTrace(new Error().stack));
                 value = newValue;
             },
@@ -107,5 +108,6 @@ logOnStacktrace.injections = [
     getPropertyInChain,
     setPropertyAccess,
     hit,
+    logMessage,
     isEmptyObject,
 ];