8000 Replace link for creating CSP by Dyras · Pull Request #1564 · roots/trellis · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace link for creating CSP #1564

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

Merged
merged 3 commits into from
Apr 12, 2025
Merged

Replace link for creating CSP #1564

merged 3 commits into from
Apr 12, 2025

Conversation

Dyras
Copy link
Contributor
@Dyras Dyras commented Mar 17, 2025

Hello!

I woke up this morning and decided to add CSP to the company website. Turns out that the checks changelog 7 year old link for this project takes you to a blog now.

So I looked around online for a similar site that is free to use. I found nothing.

Eventually I found a browser extension that allows you to create CSP by browsing around your website. I just tried it and so far it works great.

If you guys have a better site or extension, feel free using that instead :)

PS. What's going on with the community forum at the moment? I don't have Discord access and the message on the forum stating it's read only temporarily is really bare bones.

@retlehs
Copy link
Member
retlehs commented Mar 17, 2025

We should update this file based off the upstream source (https://github.com/h5bp/server-configs-nginx)

@Dyras
Copy link
Contributor Author
Dyras commented Mar 28, 2025

I really thought I responded to this, oops.

How about changing the text to match this one:

https://github.com/h5bp/server-configs-nginx/blob/main/h5bp/security/content-security-policy.conf

@swalkinshaw
Copy link
Member

That works 👍

@swalkinshaw
Copy link
Member

Sorry we weren't 100% clear before. We can't replace this entire file because there's other non-CSP settings. Specifically we should preserve the file as is, but replace the CSP related comment in this one only.

@Dyras
Copy link
Contributor Author
Dyras commented Apr 2, 2025

Ah yes, seems to also have been an error from my side. So ready for the Friday to be honest.

@swalkinshaw swalkinshaw merged commit a77ff88 into roots:master Apr 12, 2025
2 of 3 checks passed
@swalkinshaw
Copy link
Member

Thanks!

@Dyras Dyras deleted the patch-1 branch April 12, 2025 19:12
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.

3 participants
0