hlxsites / merative2

Merative.com site on Franklin
https://merative.com
Apache License 2.0
2 stars 9 forks source link

fix(csp): update CSP to include Bing & clarity.ms advertising script #315

Closed proeung closed 10 months ago

proeung commented 11 months ago

Issue

Fixes - https://jira.sdlc.merative.com/browse/MERATIVE-820

Description

Changed

Design Specs

Test URLs

Testing Instruction

aem-code-sync[bot] commented 11 months ago

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed. In case there are problems, just click the checkbox below to rerun the respective action.

aem-code-sync[bot] commented 11 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
scottbridgeman commented 11 months ago

Hi @proeung - I may be jumping the gun on testing but I am seeing an error on the test URL: https://fix-csp-bing-advertising--merative2--proeung.hlx.page/

image
sharathmrft commented 10 months ago

@proeung Still error is appearing find the screenshot below

image @keith-kaplan

aem-code-sync[bot] commented 10 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 10 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 10 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
aem-code-sync[bot] commented 10 months ago
Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
proeung commented 10 months ago

@scottbridgeman @sharathmrft Just updated the header. Can you both check again?

scottbridgeman commented 10 months ago

@proeung - Looks great! That fixed it. Thank you.

image