TYPO3 Core

There is a trouble within \TYPO3\CMS\Install\Service\SetupDatabaseService::getDatabaseConfigurationFromEnvironment() method.
The port is not casted to int after fetching it from ENV. See: https://github.com/TYPO3/typo3/blob/90e286e56eb55a1e0f5888b0f8f65917d97eedf4/typo3/sysext/install/Classes/Service/SetupDatabaseService.php#L220

How to reproduce:¶

Try to run typo3 setup command with defined TYPO3_INSTALL_DB_* variables.

Proposed fix:¶

public function getDatabaseConfigurationFromEnvironment(): array
    {
        /** @var Params $envCredentials */
        $envCredentials = [];
        foreach (['driver', 'host', 'user', 'password', 'port', 'dbname', 'unix_socket'] as $value) {
            $envVar = 'TYPO3_INSTALL_DB_' . strtoupper($value);
            if (getenv($envVar) !== false) {
                $envCredentials[$value] = getenv($envVar);
                if($value === 'port') {
                    $envCredentials[$value] = (int)$envCredentials[$value];
                }
            }
        }

Rafael Kähm wrote in #note-5:

There is a trouble within \TYPO3\CMS\Install\Service\SetupDatabaseService::getDatabaseConfigurationFromEnvironment() method.
The port is not casted to int after fetching it from ENV. See: https://github.com/TYPO3/typo3/blob/90e286e56eb55a1e0f5888b0f8f65917d97eedf4/typo3/sysext/install/Classes/Service/SetupDatabaseService.php#L220

How to reproduce:¶

Try to run typo3 setup command with defined TYPO3_INSTALL_DB_* variables.

---

Proposed fix:¶

[...]

@Rafael Kähm this should be fixed with

84231: [BUGFIX] Cast port to integer when setting up the database | https://review.typo3.org/c/Packages/TYPO3.CMS/+/84231

in main and 12.4. That means, that you need to wait for the next 13 sprint release or use composer patches to fix that.