-
Notifications
You must be signed in to change notification settings - Fork 208
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.
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
feat: add security headers #14411
base: main
Are you sure you want to change the base?
feat: add security headers #14411
Conversation
I get on the homepage:
|
@anthonydillon fixed it could you try again? |
I still get a Reused connection to https://region1.google-analytics.com. Do you need to wildcard the subdomain? |
Also we need vimeo on this page: https://ubuntu-com-14411.demos.haus/16-04/azure |
@usamabinnadeem-10 and @abhigyanghosh30 could you check creds and pro store on this demo please |
@abhigyanghosh30 can you describe how you get that page? |
So to recreate the issue
It will take you to /account/checkout which is where the error occurs |
"js.zi-scripts.com", | ||
"*.mktoresp.com", | ||
"prompts.maze.co", | ||
"*.google-analytics.com", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@anthonydillon i ve already added it as a wild card. but i cant get the error. could you verify the error is still relevant with "connect-src" or some other CSP attribute?
@anthonydillon it might not related with the security headers beaceuse:
could you verify if its working for you locally on main? |
Fixed the captcha |
@abhigyanghosh30 @usamabinnadeem-10 could you give this branch another review in regards to CUE and Pro, please? |
Can somebody review this? |
The Pro and CUE shop work now. |
Done
Added security headers.
resources
Read more from here.
QA
Issue / Card
Fixes #14446, #14447, #14448, #14449, #14450, #14451, #14452