Various modules: Send beacon wrapping fix (#12236)

* #11838 send beacon fix * Update cwireBidAdapter.js * Update 33acrossAnalyticsAdapter.js --------- Co-authored-by: Marcin Komorski <marcinkomorski@Marcins-MacBook-Pro.local> Co-authored-by: Patrick McCann <patmmccann@gmail.com>
prebid · Sep 24, 2024 · 77b974d · 77b974d
1 parent 4675958
commit 77b974d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 16 deletions.
diff --git a/modules/33acrossAnalyticsAdapter.js b/modules/33acrossAnalyticsAdapter.js
@@ -5,6 +5,7 @@ import adapterManager, { coppaDataHandler, gdprDataHandler, gppDataHandler, uspD
  * @typedef {typeof import('../src/constants.js').EVENTS} EVENTS
  */
 import { EVENTS } from '../src/constants.js';
+import { sendBeacon } from '../src/ajax.js';
 
 /** @typedef {'pending'|'available'|'targetingSet'|'rendered'|'timeout'|'rejected'|'noBid'|'error'} BidStatus */
 /**
@@ -629,9 +630,7 @@ function setCachedBidStatus(auctionId, bidId, status) {
  * @param {string} endpoint URL
  */
 function sendReport(report, endpoint) {
-  // TODO FIX THIS RULES VIOLATION
-  // eslint-disable-next-line
-  if (navigator.sendBeacon(endpoint, JSON.stringify(report))) {
+  if (sendBeacon(endpoint, JSON.stringify(report))) {
     log.info(`Analytics report sent to ${endpoint}`, report);
 
     return;

diff --git a/modules/cwireBidAdapter.js b/modules/cwireBidAdapter.js
@@ -3,6 +3,7 @@ import {getStorageManager} from '../src/storageManager.js';
 import {BANNER} from '../src/mediaTypes.js';
 import {generateUUID, getParameterByName, isNumber, logError, logInfo} from '../src/utils.js';
 import {hasPurpose1Consent} from '../src/utils/gdpr.js';
+import { sendBeacon } from '../src/ajax.js';
 
 /**
  * @typedef {import('../src/adapters/bidderFactory.js').BidRequest} BidRequest
@@ -228,9 +229,7 @@ export const spec = {
         bid: bid
       }
     }
-    // TODO FIX THIS RULES VIOLATION
-    // eslint-disable-next-line prebid/no-member
-    navigator.sendBeacon(EVENT_ENDPOINT, JSON.stringify(event))
+    sendBeacon(EVENT_ENDPOINT, JSON.stringify(event))
   },
 
   onBidderError: function (error, bidderRequest) {
@@ -242,9 +241,7 @@ export const spec = {
         bidderRequest: bidderRequest
       }
     }
-    // TODO FIX THIS RULES VIOLATION
-    // eslint-disable-next-line prebid/no-member
-    navigator.sendBeacon(EVENT_ENDPOINT, JSON.stringify(event))
+    sendBeacon(EVENT_ENDPOINT, JSON.stringify(event))
   },
 
   getUserSyncs: function(syncOptions, serverResponses, gdprConsent, uspConsent) {

diff --git a/modules/sirdataRtdProvider.js b/modules/sirdataRtdProvider.js
@@ -8,7 +8,7 @@
  * @requires module:modules/realTimeData
  */
 import adapterManager from '../src/adapterManager.js';
-import { ajax } from '../src/ajax.js';
+import { ajax, sendBeacon } from '../src/ajax.js';
 import {
   deepAccess, checkCookieSupport, deepSetValue, hasDeviceAccess, inIframe, isEmpty,
   logError, logInfo, mergeDeep
@@ -217,12 +217,7 @@ export function postContentForSemanticAnalysis(postContentToken, actualUrl) {
     if (payload && payload.length > 300 && payload.length < 300000) {
       const url = `https://contextual.sirdata.io/api/v1/push/contextual?post_content_token=${postContentToken}&url=${encodeURIComponent(actualUrl)}`;
 
-      // Use the Beacon API if supported to send the payload
-      if ('sendBeacon' in navigator) {
-        // TODO FIX RULES VIOLATION
-        // eslint-disable-next-line prebid/no-member
-        navigator.sendBeacon(url, payload);
-      } else {
+      if (!sendBeacon(url, payload)) {
         // Fallback to using AJAX if Beacon API is not supported
         ajax(url, {}, payload, {
           contentType: 'text/plain',