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

Fixed mysql authentication option causing db fail to start #26

Closed
wants to merge 1 commit into from

Conversation

RedPine404
Copy link

The --default-authentication-plugin=mysql_native_password was removed from mysql 8.4 because native-password authentication is deprecated. In mysql 8.4 it's still possible to use the native_password authentication, but the parameter is renamed to --mysql-native-password=ON

@jackyzy823
Copy link
Owner

Is this argument working on mysql 8.3 or below ? (If not , could you modify the mysql version in the compose file to like 8.4 )
Thanks


I tested the result is that it starts working from 8.2
And the old argument still exists (could we not delete this line?)


Or just pin on mysql 8.0 (since there's still a long time until EOL). Although it is not a good idea.


Note for myself; Add a warning/note to user for the upgrading of mysql version.

@RedPine404
Copy link
Author

My vote is for pinning version 8.4; this is now the default version that comes with docker tag mysql:8 but would prevent issues being introduced by future releases.

Version 8.4 is the new LTS version that was released on 10 Apr 2024 and will be supported until 30 Apr 2029.

Refer to this GitHub issue as to why the line was removed for mysql 8.4 docker-library/mysql#1048

@jackyzy823
Copy link
Owner

It looks like "--default-authentication-plugin" is deprecated but not "--authentication_policy"

@jackyzy823
Copy link
Owner

Hi , i've add a commit myself to pin mysql version to 8.4 . Thanks for your contribution!

@jackyzy823 jackyzy823 closed this May 22, 2024
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