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.

Sign up for GitHub

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

Jump to bottom

Use MySqlConnectionStringBuilder instead of 'manually' parsing the connection string #5

Open

darraghjones opened this issue Oct 19, 2020 · 1 comment

Labels

bug

darraghjones commented Oct 19, 2020 •

edited by droyad

Loading

I defined my connection string using 'initial catalogue' instead of 'database'. However, this line of code then does not work correctly:
https://github.com/DbUp/DbUp/blob/c8f94a2ff915e651b5b573328e0f01d50d8241c4/src/dbup-mysql/MySqlExtensions.cs#L27

I suggesting using the MySqlConnectionStringBuilder to parse the connection string.

darraghjones added the bug label

Member

AdrianJSClark commented Oct 19, 2020

I agree, sounds like a great idea.

Happy for you to submit a PR if you have time, @darraghjones. Otherwise someone will have a look as soon as they are free.

droyad transferred this issue from DbUp/DbUp

mjauernig mentioned this issue

Upgrade schema is failing to upgrade a new database. #6

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment