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

Support for HandshakeResponse320 #11

Merged
merged 3 commits into from
Jul 5, 2020
Merged

Support for HandshakeResponse320 #11

merged 3 commits into from
Jul 5, 2020

Conversation

rnbguy
Copy link
Contributor

@rnbguy rnbguy commented Jul 1, 2020

First effort to fix #10.

It is not complete. But it parses everything that it should return from clienthandshake().

reference to mysql client handshake protocol.

@jonhoo
Copy link
Owner

jonhoo commented Jul 2, 2020

This looks good to me! Could you merge master into this so that you get the fixed CI and we can see how the tests fare?

@rnbguy
Copy link
Contributor Author

rnbguy commented Jul 5, 2020

Done :)

@jonhoo jonhoo merged commit 9e06100 into jonhoo:master Jul 5, 2020
flaneur2020 pushed a commit to flaneur2020/msql-srv that referenced this pull request Jan 12, 2022
…MORE

Send ok packet after com field list
jonhoo pushed a commit that referenced this pull request Feb 3, 2024
Put 1.70 in there (for instance if you want to pin against OnceLock stabilizing) and it will actually test 1.7 as it appears github auto converts this to a float?

Putting in quotes seems to do the right thing here
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

Successfully merging this pull request may close these issues.

can not connect with JDBC driver
2 participants