Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/mysql schema state ssl options string #54508

Closed

Conversation

yuriizee
Copy link

@yuriizee yuriizee commented Feb 7, 2025

[11.x] Added the ability to disable secure SSL connection for DB 8.0 versions and refactoring options
Look this issue (#54269)

--ssl
--ssl-ca

If you use the RefreshDatabase trait in a test case, then when you try to migrate with the trait, you will see the error
ERROR 2026 (HY000): TLS/SSL error: SSL is required, but the server does not support it
Tested on mysql version 8.0-8.0.35

Since SSL is required by default for an outdated client

This fix allows you to skip SSL server verification, and successfully run mysql commands (including during migration in tests)

The skip parameter will be added only if env MYSQL_ATTR_SSL_VERIFY_SERVER_CERT=OFF is specified in the database configuration.
Otherwise, the mysql connection string will remain unchanged.

@taylorotwell
Copy link
Member

Thanks for your pull request to Laravel!

Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.

If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!

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