Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[13.x] Fixes PDO::MYSQL_ATTR_SSL_CA deprecation messages#57287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@SanderMuller
Copy link
Contributor

@SanderMullerSanderMuller commentedOct 7, 2025
edited
Loading

⚠️ 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 sincePdo\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.phpon line 84
@github-actions
Copy link

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 commentedOct 7, 2025
edited
Loading

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 forsymfony/polyfill#547

@SanderMuller
Copy link
ContributorAuthor

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 reacted with thumbs up emoji

@SanderMuller
Copy link
ContributorAuthor

@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
ContributorAuthor

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

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SanderMuller@crynobone

[8]ページ先頭

©2009-2025 Movatter.jp