-
Notifications
You must be signed in to change notification settings - Fork 254
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
Issue #265 - _process_auth implementation #266
Conversation
Codecov Report
@@ Coverage Diff @@
## master #266 +/- ##
=======================================
Coverage 92.91% 92.91%
=======================================
Files 9 9
Lines 1157 1157
Branches 172 172
=======================================
Hits 1075 1075
Misses 53 53
Partials 29 29 Continue to review full report at Codecov.
|
Looks good, but we need test case... |
Do you know how to setup mysql server to support that auth? We can build docker image and test agains it. |
I'm working on test cases. During next week I will update this pr. My mysql version is 5.7.20, but I think it can be easeally reproduced on lower mysql version, because MySQL announce the AuthSwitch packet which contains two component |
@jettify I added test that can easily reproduce the issue locally with docker |
Implementation merged to master with this PR #280 |
Thanks a lot! |
No description provided.