[UserId] - ID5 - Fixed case when consentData is undefined (No CMP) (p…

…rebid#4215)
trchandraprakash · Sep 20, 2019 · f1642f1 · f1642f1
1 parent 1e03b4b
commit f1642f1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/modules/id5IdSystem.js b/modules/id5IdSystem.js
@@ -38,7 +38,7 @@ export const id5IdSubmodule = {
       utils.logError(`User ID - ID5 submodule requires partner to be defined as a number`);
       return undefined;
     }
-    const hasGdpr = (typeof consentData.gdprApplies === 'boolean' && consentData.gdprApplies) ? 1 : 0;
+    const hasGdpr = (consentData && typeof consentData.gdprApplies === 'boolean' && consentData.gdprApplies) ? 1 : 0;
     const gdprConsentString = hasGdpr ? consentData.consentString : '';
     const storedUserId = this.decode(cacheIdObj);
     const url = `https://id5-sync.com/g/v1/${configParams.partner}.json?1puid=${storedUserId ? storedUserId.id5id : ''}&gdpr=${hasGdpr}&gdpr_consent=${gdprConsentString}`;

diff --git a/modules/userId/index.js b/modules/userId/index.js
@@ -12,7 +12,7 @@
  * @summary performs action to obtain id and return a value in the callback's response argument
  * @name Submodule#getId
  * @param {SubmoduleParams} configParams
- * @param {ConsentData} consentData
+ * @param {ConsentData|undefined} consentData
  * @param {(Object|undefined)} cacheIdObj
  * @return {(Object|function)} id data or a callback, the callback is called on the auction end event
  */