Skip to content
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

Switch to report only for CSP #7116

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Switch to report only for CSP #7116

merged 1 commit into from
Jan 29, 2024

Conversation

sainak
Copy link
Member

@sainak sainak commented Jan 29, 2024

This pull request switches the Content-Security-Policy header to Content-Security-Policy-Report-Only in order to report violations without blocking any content.

@sainak sainak requested a review from a team as a code owner January 29, 2024 09:18
Copy link

vercel bot commented Jan 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2024 9:29am

Copy link

netlify bot commented Jan 29, 2024

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 0588b60
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/65b76fdd76abe40008b05185
😎 Deploy Preview https://deploy-preview-7116--care-egov-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

cypress bot commented Jan 29, 2024

Passing run #1980 ↗︎

0 104 0 0 Flakiness 0

Details:

Switch to report only for CSP
Project: CARE Commit: 0588b60599
Status: Passed Duration: 02:50 💡
Started: Jan 29, 2024 9:32 AM Ended: Jan 29, 2024 9:35 AM

Review all test suite changes for PR #7116 ↗︎

@rithviknishad rithviknishad added the reviewed reviewed by a core member label Jan 29, 2024
@nihal467
Copy link
Member

LGTM

@nihal467 nihal467 linked an issue Jan 29, 2024 that may be closed by this pull request
2 tasks
@khavinshankar khavinshankar merged commit c154e7a into develop Jan 29, 2024
62 checks passed
Copy link

@sainak We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.

@khavinshankar khavinshankar deleted the sainak/fix/csp branch January 29, 2024 14:19
@cypress cypress bot mentioned this pull request Jan 30, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewed reviewed by a core member tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSP related bug
5 participants