-
Notifications
You must be signed in to change notification settings - Fork 131
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
enable ssl_version as a JKG config option #340
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @applecool. I just had a few comments.
@kevin-bates Please take a look :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking good and testing fine. Just had the comments regarding dependency on the other items.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @applecool - thank you!
If you're so inclined, we could certainly use this change in Enterprise Gateway as well!
Thank you :) Should I create the issue on that repo or can I just reference the above issue in that repo? |
Referencing the issue in a pull request is fine. Thank you!! |
Thanks for your contributions @applecool - I'll try to cut 2.4.3 by end of day tomorrow. |
Awesome. Thank you so much. Really appreciate it :) |
JKG 2.4.3 is now up on pypi. |
Hey @kevin-bates Is it also available on conda |
It should be on conda-forge although I just now merged the PR. |
Sweet. I can see it on |
Fixes #339