search

hashicorp / terraform-provider-mysql

Terraform MySQL provider – This Terraform provider is archived per our provider archiving process: https://terraform.io/docs/internals/archiving.html
https://www.terraform.io/docs/providers/mysql/
Mozilla Public License 2.0
61 stars 189 forks source link

Initial TLS support #17

Closed bonifaido closed 6 years ago

bonifaido commented 7 years ago

Fixes: #7

bonifaido commented 7 years ago

Ready for review.

bonifaido commented 7 years ago

Can I request a review on this one please?

bonifaido commented 7 years ago

cc @apparentlymart

xanonid commented 6 years ago

It would be very helpful if this could be merged and released soon. Is there a way to support the progress of this pull request?

bonifaido commented 6 years ago

I would be more than happy to be able to get this merged.

digilist commented 6 years ago

@Gerifield @sagikazarmark You both approved this PR. Is there any reason that it cannot be merged? Otherwise it would be really really great to see this merged! 🙂

sagikazarmark commented 6 years ago

We use this patch in production for quite some time now, works fine.

bonifaido commented 6 years ago

@radeksimko can we get this merged?

bonifaido commented 6 years ago

Hi @justincampbell, I saw you are the only publicly visible member of this GitHub organization, probably you can help us merge this PR.

justincampbell commented 6 years ago

@bonifaido Although I do work on some providers, I'm not familiar with the mysql one, sorry! I'll ask around internally though.

bonifaido commented 6 years ago

Any updates on this one @justincampbell ?

bigdot commented 6 years ago

guys, any updates on this one ?

silvanu commented 6 years ago

Hi @tombuildsstuff, Could you take a look on this PR and merge it?

joestump commented 6 years ago

@bonifaido thanks for your PR and sorry for the delay in getting it merged! I was just added as a maintainer and will be trying to get all of these PRs reviewed and merged. This looks like a good one to get wrapped up!

joestump commented 6 years ago

Closing in favor of #43.