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

Update missing-sri.yaml with css checks #11338

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

amarsct
Copy link

@amarsct amarsct commented Dec 12, 2024

Fixes #11337

Template / PR Information
The missing-sri template has been added in PR #8079 but it has an issue that it doesn't identify missing-sri in "link" HTML tags (e.g, in fonts loading, css)

References:
https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity
Template Validation
I've validated this template locally?
YES

@GeorginaReeder
Copy link

Thanks for your contribution @amarsct !

We also have a Discord server, which you’re more than welcome to join. It's a great place to connect with fellow contributors and stay updated with the latest developments!

Ignores rel attribute that isn't stylesheets
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.

[FALSE-NEGATIVE] ...
2 participants