Skip to content

Conversation

@SanderMuller
Copy link
Contributor

@SanderMuller SanderMuller commented Oct 7, 2025

⚠️ Work in progress

PDO::MYSQL_ATTR_SSL_CA throws deprecation messages as of PHP 8.5:

Deprecated
: Constant PDO::MYSQL_ATTR_SSL_CA is deprecated since 8.5, use Pdo\Mysql::ATTR_SSL_CA instead in /Users/sandermuller/Documents/GitHub/hihaho/vendor/laravel/framework/config/database.php on line 84

Laravel 13 is expected to be released somewhere between February and march 2026. PHP 8.3 active support ends on 31 dec 2024. With this PR I propose to drop PHP 8.3 support by then so we can start fixing PHP 8.5 deprecation errors since Pdo\Mysql is only available as of PHP 8.4

fixes #57141

Laravel 13 is expected to be released somewhere between February and march 2026. PHP 8.3 active support ends on 31 dec 2024.

PDO::MYSQL_ATTR_SSL_CA throws deprecation errors as of PHP 8.5:
Deprecated
: Constant PDO::MYSQL_ATTR_SSL_CA is deprecated since 8.5, use Pdo\Mysql::ATTR_SSL_CA instead in
/Users/sandermuller/Documents/GitHub/hihaho/vendor/laravel/framework/config/database.php
on line 84
@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Thanks for submitting a PR!

Note that draft PR's are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

@crynobone
Copy link
Member

crynobone commented Oct 7, 2025

We're not removing PHP 8.2 and 8.3 on patch release.

@crynobone
Copy link
Member

The best way forward is actually to wait for symfony/polyfill#547

@SanderMuller
Copy link
Contributor Author

We're not removing PHP 8.2 and 8.3 on patch release.

I understand, that's why it's a PR to master for 13.x, though we can probably indeed wait for the polyfil and then make this change in 12.x

@crynobone
Copy link
Member

Laravel 13 will requires PHP 8.3 as minimum

@SanderMuller
Copy link
Contributor Author

@crynobone will close if that's a hard decision (understandable one, since it has security fixes support for ~a year from the release of L13)

@SanderMuller
Copy link
Contributor Author

Now waiting for symfony/polyfill#549 to recreate this PR based on that polyfill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants