DigiTrust Facade init GH Issue 3911

modules/digiTrustIdSystem.js

@@ -9,11 +9,13 @@
  * @requires module:modules/userId
  */
 
-// import { config } from 'src/config';
 import * as utils from '../src/utils'
 import { ajax } from '../src/ajax';
 import { submodule } from '../src/hook';
-// import { getGlobal } from 'src/prebidGlobal';
+import { attachIdSystem } from '../modules/userId/index';
+
+var fallbackTimeout = 1550; // timeout value that allows userId system to execute first
+var fallbackTimer = 0; // timer Id for fallback init so we don't double call
 
 /**
  * Checks to see if the DigiTrust framework is initialized.
@@ -81,7 +83,7 @@ function writeDigiId(id) {
   var date = new Date();
   date.setTime(date.getTime() + 604800000);
   var exp = 'expires=' + date.toUTCString();
-  document.cookie = key + '=' + encId(id) + '; ' + exp + '; path=/;';
+  document.cookie = key + '=' + encId(id) + '; ' + exp + '; path=/;SameSite=none;';
 }
 
 /**
@@ -90,6 +92,10 @@ function writeDigiId(id) {
  */
 function initDigitrustFacade(config) {
   var _savedId = null; // closure variable for storing Id to avoid additional requests
+
+  clearTimeout(fallbackTimer);
+  fallbackTimer = 0;
+
   var facade = {
     isClient: true,
     isMock: true,
@@ -162,6 +168,10 @@ function initDigitrustFacade(config) {
     }
   }
 
+  if (config && isFunc(config.callback)) {
+    facade._internals.initCallback = config.callback;
+  }
+
   if (window && window.DigiTrust == null) {
     window.DigiTrust = facade;
   }
@@ -306,11 +316,12 @@ var testHook = {};
  * Exposes the test hook object by attaching to the digitrustIdModule.
  * This method is called in the unit tests to surface internals.
  */
-function surfaceTestHook() {
-  digitrustIdModule['_testHook'] = testHook;
+export function surfaceTestHook() {
+  digiTrustIdSubmodule['_testHook'] = testHook;
+  return testHook;
 }
 
-testHook.initDigitrustFacade = initDigitrustFacade;
+testHook.initDigitrustFacade = initDigitrustFacade; // expose for unit tests
 
 /** @type {Submodule} */
 export const digiTrustIdSubmodule = {
@@ -336,4 +347,19 @@ export const digiTrustIdSubmodule = {
   _testInit: surfaceTestHook
 };
 
+// check for fallback init of DigiTrust
+function fallbackInit() {
+  if (resultHandler.retryId == 0 && !isInitialized()) {
+    // this triggers an init
+    var conf = {
+      member: 'fallback',
+      callback: noop
+    };
+    getDigiTrustId(conf);
+  }
+}
+
+fallbackTimer = setTimeout(fallbackInit, fallbackTimeout);
+
+attachIdSystem(digiTrustIdSubmodule);
 submodule('userId', digiTrustIdSubmodule);

modules/digiTrustIdSystem.md

@@ -24,14 +24,17 @@ for further instructions.
 								site: 'example_site_id'
 							},
 							callback: function (digiTrustResult) {
-							// This callback method is optional
+							// This callback method is optional and used for error handling
+							// in many if not most cases.
+							/*
 								if (digiTrustResult.success) {
 									// Success in Digitrust init;
 									// 'DigiTrust Id (encrypted): ' + digiTrustResult.identity.id;
 								}
 								else {
 									// Digitrust init failed
 								}
+							*/
 							}
 						},
 						storage: {

test/spec/modules/digitrustIdSystem_spec.js

@@ -0,0 +1,50 @@
+import {
+  digiTrustIdSubmodule,
+  surfaceTestHook
+} from 'modules/digiTrustIdSystem';
+
+let assert = require('chai').assert;
+let expect = require('chai').expect;
+
+var testHook = null;
+
+describe('DigiTrust Id System', function () {
+  it('Should create the test hook', function (done) {
+    testHook = surfaceTestHook();
+    assert.isNotNull(testHook, 'The test hook failed to surface');
+    var conf = {
+      init: {
+        member: 'unit_test',
+        site: 'foo'
+      },
+      callback: function (result) {
+      }
+    };
+    testHook.initDigitrustFacade(conf);
+    window.DigiTrust.getUser(conf);
+    expect(window.DigiTrust).to.exist;
+    expect(window.DigiTrust.isMock).to.be.true;
+    done();
+  });
+
+  it('Should report as client', function (done) {
+    delete window.DigiTrust;
+    testHook = surfaceTestHook();
+
+    var conf = {
+      init: {
+        member: 'unit_test',
+        site: 'foo'
+      },
+      callback: function (result) {
+        expect(window.DigiTrust).to.exist;
+        expect(result).to.exist;
+        expect(window.DigiTrust.isMock).to.be.true;
+      }
+    };
+    testHook.initDigitrustFacade(conf);
+    expect(window.DigiTrust).to.exist;
+    expect(window.DigiTrust.isClient).to.be.true;
+    done();
+  });
+});