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

Request: Update MySql.Data.dll to support clear text password authentication #447

Closed
coryreid opened this issue May 13, 2022 · 4 comments
Closed

Comments

@coryreid
Copy link

I was working on using RoundhousE to perform MySQL migrations using Azure Managed Identities. I got the process working to connect, but when you pass the token, it requires the MySQL connection to accept clear text passwords.

When running through RoundhousE, I get the following error reported back in the stack trace:

MySql.Data.MySqlClient.MySqlException (0x80004005): Authentication method 'mysql_clear_password' not supported by any of the available plugins.

It looks like RoundhousE is using MySQL.Data.Dll version 8.0.16, and support for cleartext password authentication was added in version 8.0.22 from what I can see.

I had a difficult time getting RoundhousE running locally, otherwise I would've pushed this into a PR and run the testing to ensure no compatibility issues. I also know that Grate is the primary focus, but would love the opportunity to utilize managed identity authentication for cloud database providers using RoundhousE if it's possible to get the dll bumped.

@twerthi
Copy link

twerthi commented May 21, 2022

+1, the cleartext password authentication would enable this to be used with Azure, AWS, and GCP native authentication providers as they all implement similar methods.

@erikbra
Copy link
Member

erikbra commented Jun 5, 2022

Hi, guys, just a quick note that no one is actively maintaining RoundhousE at the moment. I was one of the main maintainers for a couple of years, but I did a re-implementation using more modern tools, called grate. Please check out #438 for more info

@coryreid
Copy link
Author

coryreid commented Jun 6, 2022

Hey @erikbra , thanks for the response. Will look at Grate to support more of the cloud native authentication workflows, I can see where you're using MySqlConnector instead of the traditional Data.dll that should support what we're needing! 🤞

@coryreid coryreid closed this as not planned Won't fix, can't repro, duplicate, stale Jun 6, 2022
@erikbra
Copy link
Member

erikbra commented Jun 11, 2022

Please do look at Grate, and feel free to file a similar issue there, if there is a screnario we don't support out-of-the box there. The aim of starting a rewrite was partly to be able to use new and modern libraries, supporting more functionality, without being gagged and bound by ancient dependencies :)

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

No branches or pull requests

3 participants