-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add Content Security Policy headers to all jQuery content sites #54
Comments
Prevent future regressions. Ref #3. Ref jquery/infrastructure-puppet#54
This also depends on jquery/jquery-wp-content#463 Also, the nginx changes are only being deployed to staging atm. |
@timmywil Of the three changed roles, only grunt has staging. It seems https://stage.gruntjs.com/ is now down. I guess an nginx syntax error? |
After consulting the docs, I don't see anything obviously wrong with the syntax. Instead, I think the issue has to do with the grunt site's use of |
@timmywil That didn't seem to bring the site back. I tried logging into the droplet, to check its puppet log and nginx error, but it's not responding to SSH. Looks like something on 22 Aug (two days before your first patch). Could it be a coincidence? |
I've rebooted the instance and the site is now back up. Investigation at #60 (unrelated to this). |
- disable style tag added in WordPress 6.7 Ref jquery/infrastructure-puppet#54 Closes gh-473
Remaining items:
Then we can finally call this done, but continue watching logs for anything I missed. |
- allow scripts, styles, and images from code.jquery.com Ref jquery/infrastructure-puppet#54 Closes gh-8
We can finally call this done! We'll want to double check the CSP report logs when the blogs are switched to jquery-wp-content, but no CSP exceptions should need to be added for the sake of the blog sites. The typekit exceptions won't apply because jquery-wp-content uses a self-hosted font and Timo and I migrated all remote images. |
Proposed header value
This should be tested with a report header first
The text was updated successfully, but these errors were encountered: