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.

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

Jump to bottom

CSP configuration #1746

Open
nmorduch opened this issue Sep 29, 2021 · 1 comment
Open

CSP configuration #1746

nmorduch opened this issue Sep 29, 2021 · 1 comment
Assignees
@chigby

Comments

@nmorduch
Copy link
Contributor

Follow up to #1731
@nmorduch nmorduch added the 2 med label Sep 29, 2021
@nmorduch nmorduch changed the title Follow up on CSP configuration [1h] Add sites to CSP configuration Sep 30, 2021
chigby added a commit that referenced this issue Sep 30, 2021
@chigby
Add CSP sources present in reports to sentry
619a05d 
Fixes #1746
@chigby chigby mentioned this issue Sep 30, 2021
Merged
@nmorduch nmorduch reopened this Sep 30, 2021
@nmorduch nmorduch removed the 2 med label Mar 31, 2023
@nmorduch nmorduch linked a pull request Mar 31, 2023 that will close this issue
Additional csp updates #1753
Merged
@nmorduch nmorduch removed a link to a pull request Mar 31, 2023
Additional csp updates #1753
Merged
@nmorduch nmorduch changed the title [1h] Add sites to CSP configuration CSP configuration Mar 31, 2023
@chigby chigby mentioned this issue Mar 31, 2023
Merged
@nmorduch
Copy link
Contributor Author

nmorduch commented Apr 4, 2023

Is there a reason some of the sources don't list the protocol at all? Ideally they'd all be https, but even if we have to add some http in there, I'd rather not leave them unspecified if we don't have to.

@chigby chigby mentioned this issue Apr 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chigby chigby

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chigby @nmorduch