Skip to content

Please review and apply the patches #3

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Please review and apply the patches #3

wants to merge 2 commits into from

Conversation

@mnikhil-git
Copy link

@mnikhil-git mnikhil-git commented Jan 2, 2012

Hi Tsucchi,

I have compiled the patches into my copy of the repository for 2 issues, I had opened with your project.
Please review the differences and pull into mainstream as you see fit.

The solution patch seems to be working for my needs and so far have not observed any issues.

Thanks,
Nikhil

@mnikhil-git
Support for extended MySQL authentication for servers above 4.2 (inc…
75da5c2 
…ludes 5.x)

 read the end of the packet stream correctly.
  -- Krisodb's patch.
  Details found at tsucchi#1
@mnikhil-git
Read the end of the packet correctly for the record set query results.
f658b9b 
Net::MySQL hangs indefinitely waiting for another packet while the stream is already ended
for the single query result set messages from the server.
Details found at tsucchi#2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mnikhil-git