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

Add checks to block PRs merging changes which include 'mystagingwebsite.com' #8253

Open

dsas opened this issue Oct 9, 2024 · 1 comment

Contributor

dsas commented Oct 9, 2024

I want to prevent #8252 from reoccurring - people committing links to their development site, or embedding resources from them.

Arguably we should block all external links outside of a small allowlist...

Contributor

alaczek commented Oct 15, 2024

Yes, plus 100! We might also want to check for local links, e.g. http://localhost:xxxx/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment