make sure we only register once

Author: ssundahlTTD

src/integrationTests/secureSignal.test.ts

@@ -124,7 +124,7 @@ describe('Secure Signal Tests', () => {
         __uid2SSProviderScriptLoad();
         __uid2InternalHandleScriptLoad();
         (window.__uid2 as UID2).init({});
-        expect(secureSignalProvidersPushMock).toHaveBeenCalled();
+        expect(secureSignalProvidersPushMock).toHaveBeenCalledTimes(1);
         await expect(secureSignalProvidersPushMock).toHaveBeenCalledWith(
           expect.objectContaining({
             id: 'uidapi.com',
@@ -141,7 +141,7 @@ describe('Secure Signal Tests', () => {
         __uid2InternalHandleScriptLoad();
         (window.__uid2 as UID2).init({});
         (window.__uid2 as UID2).setIdentity(identity);
-        expect(secureSignalProvidersPushMock).toHaveBeenCalled();
+        expect(secureSignalProvidersPushMock).toHaveBeenCalledTimes(1);
         await expect(secureSignalProvidersPushMock).toHaveBeenCalledWith(
           expect.objectContaining({
             id: 'uidapi.com',
@@ -157,8 +157,7 @@ describe('Secure Signal Tests', () => {
         __uid2InternalHandleScriptLoad();
         (window.__uid2 as UID2).init({ identity });
         __uid2SSProviderScriptLoad();
-        // will fire on both InitCompleted and SdkLoaded
-        expect(secureSignalProvidersPushMock).toHaveBeenCalledTimes(2);
+        expect(secureSignalProvidersPushMock).toHaveBeenCalledTimes(1);
         await expect(secureSignalProvidersPushMock).toHaveBeenCalledWith(
           expect.objectContaining({
             id: 'uidapi.com',

src/secureSignal_shared.ts

@@ -4,10 +4,12 @@ const MAXIMUM_RETRY = 3;
 export class UidSecureSignalProvider implements UidSecureSignalProviderType {
   debug: boolean;
   isEuid: boolean;
+  hasRegisteredSecureSignals: boolean;
 
   constructor(debug = false, isEuid = false) {
     this.debug = debug;
     this.isEuid = isEuid;
+    this.hasRegisteredSecureSignals = false;
 
     if (
       (!this.isEuid && typeof window.getUid2AdvertisingToken === 'function') ||
@@ -31,15 +33,19 @@ export class UidSecureSignalProvider implements UidSecureSignalProviderType {
     };
 
     window.googletag.secureSignalProviders = window.googletag.secureSignalProviders || [];
-    window.googletag.secureSignalProviders.push({
-      id: this.isEuid ? 'euid.eu' : 'uidapi.com',
-      collectorFunction: async () => {
-        this.logging('collectorFunction invoked');
-        const uidAdvertisingToken = await this.getUidAdvertisingTokenWithRetry(uidHandler);
-        this.logging(`collectorFunction pushes: ${uidAdvertisingToken}`);
-        return uidAdvertisingToken;
-      },
-    });
+
+    if (!this.hasRegisteredSecureSignals) {
+      window.googletag.secureSignalProviders.push({
+        id: this.isEuid ? 'euid.eu' : 'uidapi.com',
+        collectorFunction: async () => {
+          this.logging('collectorFunction invoked');
+          const uidAdvertisingToken = await this.getUidAdvertisingTokenWithRetry(uidHandler);
+          this.logging(`collectorFunction pushes: ${uidAdvertisingToken}`);
+          return uidAdvertisingToken;
+        },
+      });
+    }
+    this.hasRegisteredSecureSignals = true;
   };
 
   public logging = (message: string) => {