Defaultgdprscope (#657)

* changes to handle gdpr null case * GDPR namespace undefined check added * undefined check in consent management * Logs added for privacy gdpr checks * Used Template literals --------- Co-authored-by: Manasi Moghe <[email protected]>
PubMatic-OpenWrap · May 2, 2023 · a7124c4 · a7124c4
1 parent 88084ab
commit a7124c4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/modules/userId/index.js b/modules/userId/index.js
@@ -1068,9 +1068,9 @@ function initSubmodules(dest, submodules, consentData, forceRefresh = false) {
 
     // another consent check, this time each module is checked for consent with its own gvlid
     let { userIdModules, hasValidated } = validateGdprEnforcement(submodules, consentData);
-    logWarn("Privacy - Logging userIds allowed for syncup - "+userIdModules);
-    logWarn("Privacy - Logging hasValidated flag - "+hasValidated);
-    logWarn("Privacy - Logging consentData received - "+consentData);
+    logWarn(`Privacy - Logging userIds allowed for syncup - ${userIdModules}`);
+    logWarn(`Privacy - Logging hasValidated flag - ${hasValidated}`);
+    logWarn(`Privacy - Logging consentData received - ${consentData}`);
     if (!hasValidated && !hasPurpose1Consent(consentData)) {
       logWarn(`${MODULE_NAME} - gdpr permission not valid for local storage or cookies, exit module`);
       return [];

diff --git a/src/utils/gpdr.js b/src/utils/gpdr.js
@@ -7,13 +7,13 @@ import {deepAccess, logWarn} from '../utils.js';
  * @returns {boolean} true if the gdprConsent is null-y; or GDPR does not apply; or if purpose 1 consent was given.
  */
 export function hasPurpose1Consent(gdprConsent) {
-  logWarn("Privacy - checking purpose1Consent - "+gdprConsent);
+  logWarn(`Privacy - checking purpose1Consent - ${gdprConsent}`);
   if(gdprConsent === null) {
-    logWarn("Privacy - gdprConsent is null, checking value of defaultGdprScope = "+owpbjs?.getConfig().consentManagement?.gdpr?.defaultGdprScope);
+    logWarn(`Privacy - gdprConsent is null, checking value of defaultGdprScope = ${owpbjs?.getConfig().consentManagement?.gdpr?.defaultGdprScope}`);
     return !(owpbjs?.getConfig().consentManagement?.gdpr?.defaultGdprScope === true)
   }
   if (gdprConsent?.gdprApplies) {
-    logWarn("Privacy - gdprConsent?.gdprApplies = "+gdprConsent?.gdprApplies+" and purpose consent = "+gdprConsent.vendorData.purpose.consents);
+    logWarn(`Privacy - gdprConsent?.gdprApplies = ${gdprConsent?.gdprApplies} and purpose consent = ${gdprConsent.vendorData.purpose.consents}`);
     return deepAccess(gdprConsent, 'vendorData.purpose.consents.1') === true;
   }
   return true;