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

Update README.md with caching_sha2_password limitation #141

Closed
wants to merge 3 commits into from

Conversation

enortham
Copy link

Add caching_sha2_password limitation. I lost a lot of time trying to figure out why I couldn't authenticate and I think this should be in the README to save others in the future. Hopefully this will be fixed soon based on #26.

Add caching_sha2_password limitation.
@enortham enortham changed the title Update README.md Update README.md with caching_sha2_password limitation Dec 13, 2023
@andyjeffries
Copy link

I would change this to comment that it's the default configuration in the official Docker images too, otherwise people may not be aware of the setting and just think "I'm OK, I didn't change that option so I should be fine" and cause lots of wasted time.

Updated to include that it's the sha2 plugin is default on MySQL 8.
@enortham
Copy link
Author

Thanks @andyjeffries. I added that it's the default for MySQL 8 which should affect not only Docker but anything that uses 8.

@bensheldon
Copy link
Contributor

Thank you for proposing this change! I made a few tweaks and will accept 👍🏻

@bensheldon bensheldon added the documentation Improvements or additions to documentation label Jan 24, 2024
@andyjeffries
Copy link

Hi @bensheldon you removed the part about it being default for MySQL 8 and for the MySQL 8 docker build? That feels like key information.

@andyjeffries
Copy link

Of course, this is something that @eileencodes is working on, so soon (for some definition...) this documentation change won't be necessary anyway.

Appreciate your work (all of you) on this great library, really looking forward to switching properly to it.

@eileencodes
Copy link
Member

Closing in favor of #165

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants