Skip to content

Fix Microsoft Clarity consent API property names for GDPR compliance #786

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Sep 21, 2025
Merged

Fix Microsoft Clarity consent API property names for GDPR compliance #786

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1fc38dc
Initial plan
Copilot Sep 20, 2025
f82dd9a
Fix Microsoft Clarity consent API property names and add verification…
Copilot Sep 20, 2025
5a466de
Delete EssentialCSharp.Web/wwwroot/consent-test.html
BenjaminMichaelis Sep 21, 2025
dbe1456
Update EssentialCSharp.Web/wwwroot/js/consent-manager.js
BenjaminMichaelis Sep 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 48 additions & 2 deletions EssentialCSharp.Web/wwwroot/js/consent-manager.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -312,8 +312,8 @@ class ConsentManager {
if (window.clarity) {
try {
clarity('consentv2', {
ad_Storage: this.consentState.ad_storage,
analytics_Storage: this.consentState.analytics_storage
ad_storage: this.consentState.ad_storage,
analytics_storage: this.consentState.analytics_storage
});
} catch (error) {
console.warn('Failed to update Clarity consent:', error);
Expand Down Expand Up @@ -507,4 +507,50 @@ window.openConsentPreferences = function() {
if (window.consentManager) {
window.consentManager.openConsentPreferences();
}
};

// Global function for verifying consent implementation (testing/debugging)
window.verifyClarityConsent = function() {
console.log('=== Microsoft Clarity Consent Verification ===');

if (typeof clarity === 'undefined') {
console.log('❌ Clarity not loaded yet. Please wait and try again.');
return;
}

try {
clarity('metadata', (d, upgrade, consent) => {
console.log('✅ Clarity metadata callback executed');
console.log('📊 Consent Status:', consent);

if (consent) {
const analyticsStatus = consent.analytics_storage || 'UNKNOWN';
const adStatus = consent.ad_storage || 'UNKNOWN';

console.log(`🔍 Analytics Storage: ${analyticsStatus}`);
console.log(`🔍 Ad Storage: ${adStatus}`);

if (analyticsStatus === 'DENIED' && adStatus === 'DENIED') {
console.log('✅ Consent properly denied - Clarity should not set cookies');
} else if (analyticsStatus === 'GRANTED' || adStatus === 'GRANTED') {
console.log('✅ Consent granted - Clarity can set cookies based on granted permissions');
} else {
console.log('⚠️ Unexpected consent status detected');
}
} else {
console.log('❌ No consent object returned');
}

// Also show current consent manager state for comparison
if (window.consentManager) {
console.log('📋 Current Consent Manager State:');
console.log(' - Analytics Consent:', window.consentManager.hasAnalyticsConsent());
console.log(' - Advertising Consent:', window.consentManager.hasAdvertisingConsent());
console.log(' - Requires Consent:', window.consentManager.requiresConsent);
console.log(' - Full State:', window.consentManager.consentState);
}
}, false, true, true);
} catch (error) {
console.log('❌ Error running verification:', error);
}
};
Loading