Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Fixed default route path #782

Closed
wants to merge 1 commit into from
Closed

Conversation

florianv
Copy link
Contributor

When using the Apache option MultiViews the path /app will be resolved to the /app.php file by Apache. So /app/example will resolve to /app.php/example and the application will respond with a 404 because the /example route does not exist.

This PR changes the example route to /home as it was also proposed in https://github.com/symfony/symfony-standard/issues/758.

@Tobion
Copy link
Contributor

Tobion commented Feb 17, 2015

/app/example will resolve to /app.php/example

Really? According to http://httpd.apache.org/docs/2.4/content-negotiation.html I would assume it would only look for files /app/example.* which does not exist. So it should not change the path.

@florianv
Copy link
Contributor Author

Really? According to http://httpd.apache.org/docs/2.4/content-negotiation.html I would assume it would only look for files /app/example.* which does not exist. So it should not change the path.

Yes, it starts matching from the left and move to the next segment while the directory exists. So in this case it stops at app.php because the file exists.

@Tobion
Copy link
Contributor

Tobion commented Feb 17, 2015

I see, because Apache also serves files at the beginning of a path, even when they have any path segment followed.

@florianv
Copy link
Contributor Author

I see, because Apache also serves files at the beginning of a path, even when they have any path segment followed.

Yes, exactly. This PR is now superseded by this one #784 but it can also be merged as it simplifies the path to the homepage

@fabpot fabpot closed this Feb 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants