Skip to content
Snippets Groups Projects
Commit 1a861b7a authored by Helmut Hummel's avatar Helmut Hummel
Browse files

[BUGFIX] Unify PATH_thisScript with PATH_site 

In case we accept TYPO3_PATH_ROOT from the environment,
we must ensure that all paths (in particular the entry script path)
are based on this base path.

We do so, by removing the calculated path prefix with
the base path injected from the environment.

We cannot rely on paths using DIRECTORY_SEPARATOR as this depends on how
the Apache and the DocumentRoot are configured. This patch converts all
paths using forward slashes before any calculation is done.

Resolves: #80634
Releases: master
Change-Id: I3edecadb2f82e763ad414cf23a7048bac7c85556
Reviewed-on: https://review.typo3.org/52295


Reviewed-by: default avatarNicole Cordes <typo3@cordes.co>
Tested-by: default avatarNicole Cordes <typo3@cordes.co>
Tested-by: default avatarTYPO3com <no-reply@typo3.com>
Reviewed-by: default avatarBenjamin Franzke <bfr@qbus.de>
Tested-by: default avatarBenjamin Franzke <bfr@qbus.de>
Reviewed-by: default avatarStefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: default avatarStefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: default avatarSteffen Frese <steffenf14@gmail.com>
Tested-by: default avatarSteffen Frese <steffenf14@gmail.com>
Reviewed-by: default avatarFrans Saris <franssaris@gmail.com>
Tested-by: default avatarFrans Saris <franssaris@gmail.com>
Reviewed-by: default avatarTobi Kretschmann <tobi@tobishome.de>
Tested-by: default avatarTobi Kretschmann <tobi@tobishome.de>
Reviewed-by: default avatarThomas Maroschik <tmaroschik@dfau.de>
Tested-by: default avatarThomas Maroschik <tmaroschik@dfau.de>
Reviewed-by: default avatarKai Ole Hartwig <mail@ole-hartwig.eu>
Tested-by: default avatarKai Ole Hartwig <mail@ole-hartwig.eu>
Reviewed-by: default avatarHelmut Hummel <typo3@helhum.io>
Tested-by: default avatarHelmut Hummel <typo3@helhum.io>
parent 1ad24af2
No related branches found
No related tags found
No related merge requests found
Hide whitespace changes
Inline Side-by-side
typo3/sysext/core/Classes/Core/SystemEnvironmentBuilder.php
+ 36
− 41
View file @ 1a861b7a
......@@ -15,6 +15,7 @@ namespace TYPO3\CMS\Core\Core;
*/
use TYPO3\CMS\Core\Utility\GeneralUtility;
use TYPO3\CMS\Core\Utility\StringUtility;
/**
* Class to encapsulate base setup of bootstrap.
......@@ -147,27 +148,40 @@ class SystemEnvironmentBuilder
*/
protected static function definePaths($entryPointLevel = 0)
{
// Relative path from document root to typo3/ directory
// Hardcoded to "typo3/"
define('TYPO3_mainDir', 'typo3/');
// Absolute path of the entry script that was called
// All paths are unified between Windows and Unix, so the \ of Windows is substituted to a /
// Example "/var/www/instance-name/htdocs/typo3conf/ext/wec_map/mod1/index.php"
// Example "c:/var/www/instance-name/htdocs/typo3/index.php?M=main" for a path in Windows
$scriptPath = GeneralUtility::fixWindowsFilePath(self::getPathThisScript());
$rootPath = self::getRootPathFromScriptPath($scriptPath, $entryPointLevel);
// Check if the root path has been set in the environment (e.g. by the composer installer)
if (getenv('TYPO3_PATH_ROOT')) {
if ((TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI)
&& Bootstrap::usesComposerClassLoading()
&& StringUtility::endsWith($scriptPath, 'typo3')
) {
// PATH_thisScript is used for various path calculations based on the document root
// Therefore we assume it is always a subdirectory of the document root, which is not the case
// in composer mode on cli, as the binary is in the composer bin directory.
// Because of that, we enforce the document root path of this binary to be set
$scriptName = '/typo3/sysext/core/bin/typo3';
} else {
// Base the script path on the path taken from the environment
// to make relative path calculations work in case only one of both is symlinked
// or has the real path
$scriptName = substr($scriptPath, strlen($rootPath));
}
$rootPath = GeneralUtility::fixWindowsFilePath(getenv('TYPO3_PATH_ROOT'));
$scriptPath = $rootPath . $scriptName;
}
if (!defined('PATH_thisScript')) {
define('PATH_thisScript', self::getPathThisScript());
define('PATH_thisScript', $scriptPath);
}
// Absolute path of the document root of the instance with trailing slash
// Example "/var/www/instance-name/htdocs/"
if (!defined('PATH_site')) {
// Check if the site path has been set by the outside (e.g. dotenv or the composer installer)
if (getenv('TYPO3_PATH_ROOT')) {
$rootPath = getenv('TYPO3_PATH_ROOT');
define('PATH_site', $rootPath . '/');
} else {
define('PATH_site', self::getPathSite($entryPointLevel));
}
define('PATH_site', $rootPath . '/');
}
// Relative path from document root to typo3/ directory
// Hardcoded to "typo3/"
define('TYPO3_mainDir', 'typo3/');
// Absolute path of the typo3 directory of the instance with trailing slash
// Example "/var/www/instance-name/htdocs/typo3/"
define('PATH_typo3', PATH_site . TYPO3_mainDir);
......@@ -272,9 +286,8 @@ class SystemEnvironmentBuilder
{
if (TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI) {
return self::getPathThisScriptCli();
} else {
return self::getPathThisScriptNonCli();
}
return self::getPathThisScriptNonCli();
}
/**
......@@ -301,10 +314,6 @@ class SystemEnvironmentBuilder
$scriptPath = $_SERVER['SCRIPT_FILENAME'];
}
}
// Replace \ to / for Windows
$scriptPath = str_replace('\\', '/', $scriptPath);
// Replace double // to /
$scriptPath = str_replace('//', '/', $scriptPath);
return $scriptPath;
}
......@@ -360,33 +369,19 @@ class SystemEnvironmentBuilder
* - A Backend script: This is the case for the typo3/index.php dispatcher and other entry scripts like 'cli_dispatch.phpsh'
* or 'typo3/index.php' that are located inside typo3/ directly.
*
* @param string $scriptPath Calculated path to the entry script
* @param int $entryPointLevel Number of subdirectories where the entry script is located under the document root
* @return string Absolute path to document root of installation
*/
protected static function getPathSite($entryPointLevel)
protected static function getRootPathFromScriptPath($scriptPath, $entryPointLevel)
{
$entryScriptDirectory = self::getUnifiedDirectoryName(PATH_thisScript);
$entryScriptDirectory = dirname($scriptPath);
if ($entryPointLevel > 0) {
list($pathSite) = GeneralUtility::revExplode('/', $entryScriptDirectory, $entryPointLevel+1);
list($rootPath) = GeneralUtility::revExplode('/', $entryScriptDirectory, $entryPointLevel + 1);
} else {
$pathSite = $entryScriptDirectory;
}
return $pathSite . '/';
}
/**
* Remove file name from script path and unify for Windows and Unix
*
* @param string $absolutePath Absolute path to script
* @return string Directory name of script file location, unified for Windows and Unix without the trailing slash
*/
protected static function getUnifiedDirectoryName($absolutePath)
{
$directory = dirname($absolutePath);
if (TYPO3_OS === 'WIN') {
$directory = str_replace('\\', '/', $directory);
$rootPath = $entryScriptDirectory;
}
return $directory;
return $rootPath;
}
/**
......
typo3/sysext/core/Tests/Unit/Core/SystemEnvironmentBuilderTest.php
+ 0
− 11
View file @ 1a861b7a
......@@ -105,17 +105,6 @@ class SystemEnvironmentBuilderTest extends \TYPO3\TestingFramework\Core\Unit\Uni
$this->assertStringStartsWith($fakedAbsolutePart, $this->subject->_call('getPathThisScriptCli'));
}
/**
* @test
*/
public function getUnifiedDirectoryNameReturnsCorrectPathOnUnix()
{
$input = '/foo/bar/test.php';
$expected = '/foo/bar';
$actual = $this->subject->_call('getUnifiedDirectoryName', $input);
$this->assertEquals($expected, $actual);
}
/**
* @test
*/
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment