-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Description
There is a problem with the path info in the Request class.
Given a request to http://localhost/symfony-standard/app_dev.php, the info in the Request are as follows:
Request::getBaseUrl() returns /symfony-standard/app_dev.php (correct)
Request::getPathInfo() returns / which is wrong because it must be empty ``.
The reason is that Request::getBaseUrl() . Request::getPathInfo() must represent the real request path. Since it is lying about the real path, #3958 generates the incorrect relative path based on the wrong info.
So there is a problem when the path info should be empty but never is (it defaults to /).
I think it's this commit that should be reverted: 62d09b8
ping @fabpot