Skip to content

Commit

Permalink
Fix incorrect use of serverVersion for MariaDB
Browse files Browse the repository at this point in the history
This small change fixes 2 things:

1/ Incorrect use of `serverVersion` for MariaDB, which *must* be prefixed with `mariadb-`, see: doctrine/dbal#3083 && symfony/symfony-docs#9547 (and doctrine/dbal#2985 (comment) for the discussion)

2/ `doctrine/dbal` now supports `MariaDB >= 10.2.7`. I added the `.12` as a minor version, but we don't need to update it when platform.sh will use .13 or .14 etc.... as long as it's superior as `10.2.7` (see: https://github.com/doctrine/dbal/blob/f76bf5ef631cec551a86c2291fc749534febebf1/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php#L136)
  • Loading branch information
tristanbes authored Apr 4, 2018
1 parent cf697ec commit d9bca64
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion platformsh-flex-env.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function mapPlatformShDatabase() : void
switch ($endpoint['scheme']) {
case 'mysql':
// Defaults to the latest MariaDB version
$dbUrl .= '?charset=utf8mb4&serverVersion=10.2';
$dbUrl .= '?charset=utf8mb4&serverVersion=mariadb-10.2.12';
break;

case 'pgsql':
Expand Down

0 comments on commit d9bca64

Please sign in to comment.