Bug #57785
closedWrong SCRIPT_NAME with hhvm / fastcgi
100%
Description
In the following core files there is a check for PHP_SAPI
./typo3/sysext/core/Classes/Utility/GeneralUtility.php
./typo3/sysext/core/Classes/Core/SystemEnvironmentBuilder.php
./typo3/sysext/install/Classes/Controller/Action/Tool/ImportantActions.php
if ($cgiPath && (PHP_SAPI === 'fpm-fcgi' || PHP_SAPI === 'cgi' || PHP_SAPI === 'isapi' || PHP_SAPI === 'cgi-fcgi')) {
this check sets the path / SCRIPT_NAME correct if your hoster is using mod_fastcgi /fcgid. Otherwise the install tool or backend login redirects you to:
www.domain.com/path/to/your/webroot/typo3/backend.php instead of:
www.domain.com/typo3/backend.php
some days ago facebook introduced HHVM with fastcgi support and it's amazing fast.
so, TYPO3 should support it :-)
one step is to extend the PHP_SAPI check with "srv" (hiphop vm's php-sapi name):
if ($cgiPath && (PHP_SAPI === 'fpm-fcgi' || PHP_SAPI === 'cgi' || PHP_SAPI === 'isapi' || PHP_SAPI === 'cgi-fcgi' || PHP_SAPI === 'srv')) {
Updated by Markus Klein over 10 years ago
Hi khaefeli!
Thanks for posting this here?
Can you push a patch to our review system?
Updated by Gerrit Code Review over 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Gerrit Code Review over 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Gerrit Code Review over 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Gerrit Code Review over 10 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Gerrit Code Review over 10 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29335
Updated by Kevin Häfeli over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 70eccdea965127052456041e693157e312de0b32.