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

All: add CSP exceptions for wordpress admins #477

Merged
merged 1 commit into from
Jan 23, 2025
Merged

All: add CSP exceptions for wordpress admins #477

merged 1 commit into from
Jan 23, 2025

Conversation

timmywil
Copy link
Member

@timmywil timmywil commented Jan 23, 2025 •
edited
Loading

  • when the blogs are switched to use this repo's jquery theme, the theme will need to allow for data: images and fonts. It still doesn't need to add exceptions for typekit as the fonts will be self-hosted.
  • Also removed some redundant declarations. Anything not specified falls back to default-src, so setting self again isn't necessary.

Ref jquery/blog.jquery.com-theme#12

@timmywil
All: add CSP exceptions for wordpress admins
e793597 
- when the blogs are switched to use this repo's jquery theme,
  the theme will need to allow for data: images and fonts
@timmywil timmywil requested a review from Krinkle January 23, 2025 03:10
@timmywil timmywil merged commit a65e3b0 into jquery:main Jan 23, 2025
5 checks passed
@timmywil timmywil deleted the blogs_csp branch January 23, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Krinkle Krinkle Krinkle approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timmywil @Krinkle