fix: [PFG-3188] Fix circular reference error in adapters (#193)

* fix: [PFG-3188] Fix circular reference error when clone event arguments containing DOM nodes

* Bump fix version

modules/carbonAnalyticsAdapter.js

@@ -1,4 +1,4 @@
-import { deepAccess, generateUUID, logError } from '../src/utils.js';
+import { cloneEventArguments, deepAccess, generateUUID, logError } from '../src/utils.js';
 import { ajax } from '../src/ajax.js';
 import { getStorageManager } from '../src/storageManager.js';
 import {getGlobal} from '../src/prebidGlobal.js';
@@ -31,7 +31,8 @@ let timeLastAuctionEvent = null;
 
 export let carbonAdapter = Object.assign(adapter({analyticsHost, ANALYTICS_TYPE}), {
   track({eventType, args}) {
-    args = args ? JSON.parse(JSON.stringify(args)) : {};
+    args = cloneEventArguments(args || {});
+
     switch (eventType) {
       case EVENTS.AUCTION_END: {
         registerEngagement();

modules/hadronAnalyticsAdapter.js

@@ -53,7 +53,8 @@ let analyticsType = 'endpoint';
 
 let hadronAnalyticsAdapter = Object.assign(adapter({url: HADRON_ANALYTICS_URL, analyticsType}), {
   track({eventType, args}) {
-    args = args ? JSON.parse(JSON.stringify(args)) : {};
+    args = utils.cloneEventArguments(args || {});
+
     var data = {};
     if (!eventsToTrack.includes(eventType)) return;
     switch (eventType) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "prebid.js",
-  "version": "8.49.5",
+  "version": "8.49.6",
   "description": "Header Bidding Management Library",
   "main": "src/prebid.js",
   "scripts": {

src/utils.js

@@ -1083,3 +1083,29 @@ export function binarySearch(arr, el, key = (el) => el) {
   }
   return left;
 }
+
+const skipPatterns = [
+  /^\[object HTML.*]$/,
+  /^\[object Window.*]$/
+]
+
+export function cloneEventArguments(args) {
+  try {
+    return JSON.parse(
+      JSON.stringify(
+        args,
+        // When data has a reference to window or any html elements
+        // JSON.stringify fails with TypeError: Converting circular structure to JSON
+        // replacer function below excludes referenced html elements
+        (_key, value) => {
+          if (skipPatterns.some(pattern => pattern.test(String(value))) || typeof value === 'function') {
+            return undefined;
+          }
+          return value;
+        },
+      ),
+    );
+  } catch (e) {
+    return args;
+  }
+}