-
Notifications
You must be signed in to change notification settings - Fork 64
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
feat(csp): hashStyles option #278
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
src/types/index.ts
Outdated
@@ -6,6 +6,7 @@ import type { AllowedHTTPMethods, BasicAuth, CorsOptions, RateLimiter, RequestSi | |||
|
|||
export type Ssg = { | |||
hashScripts?: boolean; | |||
hashStyles?: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Could you also please add it to the configuration documentation page where the default config is displayed?
@vejja could you please add the recommendations I mentioned? I would love to merge this PR now so that we can plan to release it with the next RC version :) |
Hey @Baroshem : isn't it covered in the CSP section now ? |
Let me know if this looks ok? |
Nice work, thank you! :) |
Types of changes
A user must be able to skip adding hashes to the 'style-src' policy
This is because adding even one hash cancels 'unsafe-inline'
Note: this is only for SSG
Description
Checklist: