Fixed mysql_native_password
being used due to empty password
#14118
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I resolved a test failure on CircleCI with the newly released MySQL 8.4 Server. This issue is discussed in the following references:
https://externals.io/message/123253
#14112
#14113
In MySQL 8.4, the authentication plugin
mysql_native_password
is no longer loaded by default.However, mysqlnd still defaults to usingmysql_native_password
as the authentication method when the password is not set, as specified in.circleci/config.yml
.The same test passes on GitHub Actions because the password is set as follows:
https://github.com/php/php-src/blob/master/.github/workflows/push.yml#L52-L54
By aligning the test configurations in CircleCI with those in GitHub Actions, we can ensure compatibility with MySQL 8.4 Server.