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 SSL MySQL connections #36

Merged
merged 4 commits into from
Apr 26, 2018
Merged

Conversation

insom
Copy link
Contributor

@insom insom commented Jan 25, 2017

I believe that these commits will fix #31 by adding optional SSL support to mydumper and to myloader.

I also moved some connection establishment code which is used in more than one place into a common function in a common file.

This was tested locally on Linux with the Oracle MySQL client libraries.

@maxbube
Copy link
Collaborator

maxbube commented Feb 9, 2017

Hi insom,

This is great, I will perform some tests before merging this or give you feedback.

Thanks!!!!

@nkadel-skyhook
Copy link

I'd much appreciate this support as well.

@ChristopherSchultz
Copy link

+1 for this enhancement

@maxbube maxbube added this to the release 0.9.5 milestone Feb 19, 2018
@maxbube maxbube merged commit 146fc38 into mydumper:master Apr 26, 2018
@jairud
Copy link

jairud commented May 10, 2018

Hi. I see that is merged in master.

But how can I use?

I tried with params --ssl 1 or -s 1 and didn't work
--help don't show any information about ssl

I'm using version bb96344

@insom
Copy link
Contributor Author

insom commented May 11, 2018

Hi @jairud since this was merged there were commits which make building it conditional: 97438e2

I haven't built it to verify, but looking at the code you should see an entry in --help if your binary has SSL support built in.

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.

SSL connection Error
5 participants