Skip to content

MySQL 5.7 SSL configuration#227

Closed
cursor[bot] wants to merge 1 commit intofix/mysql57-ssl-errorfrom
cursor/mysql-5-7-ssl-configuration-e0b6
Closed

MySQL 5.7 SSL configuration#227
cursor[bot] wants to merge 1 commit intofix/mysql57-ssl-errorfrom
cursor/mysql-5-7-ssl-configuration-e0b6

Conversation

@cursor
Copy link

@cursor cursor bot commented Feb 19, 2026

Bare minimum self-checks

What do you think of a person who only does the bare minimum?

  • I've updated this PR with the latest code from main
  • I've done a cursory QA pass of my code locally
  • I've ensured all automated status check and tests pass
  • I've connected this PR to an issue

Pieces of flare

  • I've written a unit or functional test for my code
  • I've updated relevant documentation it my code changes it
  • I've updated this repo's README if my code changes it
  • I've updated this repo's CHANGELOG with my change unless its a trivial change (like updating a typo in the docs)

Finally

If you have any issues or need help please join the #contributors channel in the Lando slack and someone will gladly help you out!

You can also check out the coder guide.


This PR addresses a bug where the MySQL 8.0 client, when used with MySQL 5.7, would fail due to SSL verification errors against 5.7's self-signed certificates. The previous PR introduced a test for this scenario but omitted the necessary ssl-mode=PREFERRED configuration. This change adds ssl-mode=PREFERRED to the /etc/mysql/conf.d/lando.cnf file created by mysql-client-install.sh, ensuring proper SSL handling and allowing the related CI tests to pass.


The MySQL 8.0 client enforces SSL verification which fails with MySQL 5.7's
self-signed certificates. Adding ssl-mode=PREFERRED allows connections to
proceed without strict SSL verification, fixing the SSL error.
@cursor
Copy link
Author

cursor bot commented Feb 19, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

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