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

Follow best practices for CSP compliance #140

Open
GeekInTheNorth opened this issue Oct 22, 2024 · 1 comment
Open

Follow best practices for CSP compliance #140

GeekInTheNorth opened this issue Oct 22, 2024 · 1 comment

Comments

@GeekInTheNorth
Copy link

GeekInTheNorth commented Oct 22, 2024

It is best practice for both the Website UI and the CMS Backend to be covered by security headers including a CSP. Damage to a brand can be achieved by via gaining accessing the back end using phishing attacks etc and so this too should be protected.

When a Content Security Policy is used to govern the entire platform (including the CMS backend), then the CSP currently has to be opened to include the dependencies for this module. This includes multiple packages under the cdn.jsdeliver.net domain and depending how specific the CSP is, this can potentially open the entirety of the site the cdn.jsdeliver.net domain.

It would be more ideal for the assets used for the Geta Not Found Handler to be accessible purely using the 'self' source. This can be achieved by packing these libraries into the modules\_protected\Geta.NotFoundHandler.Optimizely folder and self serving them.

@nma76
Copy link

nma76 commented Nov 15, 2024

I agree to that. There's no way I'll add cdn.jsdelivr.net to our CSP, and packing these resources isn't a big task.

And also, the installation instructions should probably mention CSP since (i hope) most sites use them and it won't work at all with out adding cdn.jsdelivr.net to frame-ancestor, script-src and style-src.

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

No branches or pull requests

2 participants