Skip to content

MySQL 8, master, still can't connect. #799

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
Closed
arvenil opened this issue May 21, 2018 · 2 comments
Closed

MySQL 8, master, still can't connect. #799

arvenil opened this issue May 21, 2018 · 2 comments
Assignees
@julienschmidt
Labels
bug
Milestone
v1.4.0

Comments

@arvenil
Copy link
Contributor

arvenil commented May 21, 2018

I'm trying out to connect to MySQL 8 with support added in master (#794). What I'm doing wrong? I've made a repository to test this https://github.com/arvenil/mysql-test and you can check latest commit to master for the result https://github.com/arvenil/mysql-test/runs/1221644 (BTW new travis integration is really cool). Every database image/version passes except MySQL 8 which errors with malformed packet error: https://travis-ci.com/arvenil/mysql-test/jobs/125123277
@julienschmidt julienschmidt added this to the v1.4.0 milestone May 21, 2018
@julienschmidt
Copy link
Member

I can confirm. If I run a local mysql:8.0 docker container, I get the same error. Investigating...

@julienschmidt
Copy link
Member

My first guess is that empty passwords are the problem. If I set a root password it works fine.

@julienschmidt julienschmidt self-assigned this May 21, 2018
julienschmidt added a commit that referenced this issue May 21, 2018
@julienschmidt
Fix empty SHA2 password handling
4bcaa02 
Fixes #799
@julienschmidt julienschmidt mentioned this issue May 21, 2018
Merged
5 tasks
julienschmidt added a commit that referenced this issue May 23, 2018
@julienschmidt
Fix empty SHA2 password handling (#800)
ceae19c 
* Fix empty SHA2 password handling

Fixes #799

* add empty password test

* fix auth switch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@julienschmidt julienschmidt

Labels
bug
Projects
None yet
Milestone
v1.4.0
Development

No branches or pull requests
2 participants
@arvenil @julienschmidt