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

Update SSL Profile for AWS Serverless Aurora #1026

Merged
merged 1 commit into from
Oct 3, 2019

Conversation

coreyjv
Copy link
Contributor

@coreyjv coreyjv commented Oct 3, 2019

@sidorares
Copy link
Owner

Thanks @coreyjv !!!

While you at this, can I ask you to compare with https://github.com/mysqljs/mysql/blob/master/lib/protocol/constants/ssl_profiles.js and check if anything else is missing? ( This file is used identically between mysqljs/mysq and node-mysql2, would be good to factor out this and maybe constants to external module some time later )

@coreyjv
Copy link
Contributor Author

coreyjv commented Oct 3, 2019

@sidorares --

The Aurora Serverless certificate does not exist in the mysqljs project either. Would it be more appropriate to create another PR that syncs this project with mysqljs so it's clearer or do you want it done prior to merging this one?

@sidorares
Copy link
Owner

Won't hurt to merge this one now and if you can do another against mysqljs that would be great - cc @dougwilson

@sidorares sidorares merged commit ac3d303 into sidorares:master Oct 3, 2019
@coreyjv
Copy link
Contributor Author

coreyjv commented Oct 3, 2019

@sidorares -- I can certainly submit a PR to the mysqljs project as well.

When can I expect this merge to land in a release?

coreyjv added a commit to coreyjv/mysql that referenced this pull request Oct 3, 2019
Mirror change to this project to align with the node-mysql2 PR here: sidorares/node-mysql2#1026
@sidorares
Copy link
Owner

When can I expect this merge to land in a release?

I recently merged really big change and also planning one more small but breaking so it has to be major version bump, so I'm planning to have -alpha1 published for some short time. I'll try to do that today/tomorrow and if no complaints publish release some time next week

@coreyjv
Copy link
Contributor Author

coreyjv commented Oct 3, 2019

Awesome, thank you!

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.

2 participants