-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Amazon SSL for RDS 2015 #1001
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
Amazon SSL for RDS 2015 #1001
Conversation
See http://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.SSL.h tml#UsingWithRDS.SSL.Rotation
@winzig you beat me to it!!!! |
I'm going to release this today. |
@dougwilson run few tests manually against fresh rds instance and it works ok |
Nice! Merging now, then :) |
P.S. do you know why there are lot of different ones now? |
no. curious as well |
Cool. I'm examining the contents now to see what's inside. |
let me know what's inside as I'm going to update it in mysql2 as well |
Looks like it's a root CA and then one cert for each of their datacenters all signed by that root CA. That means potentially we only need to actually add one cert instead of 9. |
@sidorares do you have a RDS instance? Would you be willing to turn on the new certificate (step 3 in http://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.SSL.html#UsingWithRDS.SSL.Rotation) on your RDS and then try using this lib to connect using only the following cert?
|
already killed it. I'll recreate from home in ~2hrs |
Cool, I can wait :) 👍 I'm about to go to sleep, so I'll see your results in ~8 hours. |
|
Thanks, @sidorares , that's what I was afraid of. I need to fix the verification implementation here so it'll work in the future, but for now we'll just add all 9 certs. |
closes mysqljs#1001 closes mysqljs#1003 closes mysqljs#1004
See http://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.SSL.html#UsingWithRDS.SSL.Rotation