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

Updated csp policies to match ga documentation #57

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

ryangrundy7
Copy link
Contributor

Motivation and Context

Wheen looking into the Ga documentation around CSP policies, we saw that we didn't have the entire list they recommend using. This PR just adds the other values to the policy that we were missing.

What has changed

  • Updated CSP policies

How to test?

  • Run locally and make sure everything in the console looks as expected.
  • Run in the ssdc-rm-dev environment (Branch should already be there on srm-rh-ui) and make sure there's no error logs.
  • Check the GA console for the dev project and make sure you still see events coming

Links

Jira

@ryangrundy7 ryangrundy7 added the patch A non-feature change, e.g. bug or issue fix label Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

@ryangrundy7 ryangrundy7 merged commit 16ededc into main Nov 13, 2024
3 checks passed
@ryangrundy7 ryangrundy7 deleted the SDCSRM-723-update-csp-policies branch November 13, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch A non-feature change, e.g. bug or issue fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants