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

Caddy configuration don't allow sub-directory php files to execute. Fixes #20593 #20594

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

sarim
Copy link
Contributor

@sarim sarim commented Jan 22, 2025

Fixes #20593

For details please check linked issue.

@carsonbot carsonbot added this to the 6.4 milestone Jan 22, 2025
@javiereguiluz
Copy link
Member

Let's ping our Caddy expert @dunglas for a quick review. Thanks!

Copy link
Member

@dunglas dunglas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will also unlock some performance optimizations Caddy-side.

@javiereguiluz
Copy link
Member

Thanks Sarim for this contribution (and for the super nice and detailed issue that you created in #20593) and thanks Kévin for the review!

@javiereguiluz javiereguiluz merged commit 8377b29 into symfony:6.4 Jan 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants