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

server: Add more sources to the CSP #7645

Merged
merged 1 commit into from
Oct 3, 2024
Merged

server: Add more sources to the CSP #7645

merged 1 commit into from
Oct 3, 2024

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Oct 3, 2024

  • Allow dynamic loading of scripts such as Monaco via strict-dynamic
  • Allow more image sources
  • Add directives required by libsodium.js
  • Patch out an unnecessary use of eval

Related issues: buildbuddy-io/buildbuddy-internal#3911

@fmeum fmeum requested a review from bduffany October 3, 2024 11:34
* Allow dynamic loading of scripts such as Monaco via strict-dynamic
* Allow more image sources
* Add directives required by libsodium.js
* Patch out an unnecessary use of `eval`
@fmeum fmeum changed the title server: Add more source to the CSP server: Add more sources to the CSP Oct 3, 2024
@fmeum fmeum merged commit ed9ce06 into master Oct 3, 2024
14 of 15 checks passed
@fmeum fmeum deleted the fmeum/csp-fixes branch October 3, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants