-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add TLS Support #80
base: main
Are you sure you want to change the base?
Add TLS Support #80
Conversation
hazemkmammu
commented
Oct 9, 2023
- Add TLS Support by accepting TLS options and passing them to Mojo::IOLoop::Client.
@jhthorsen Hi Jan. Could you please review this change? This should add support for TLS connection to Redis. SSL/TLS is supported by Redis starting with version 6. Please let me know if you have any questions. Thanks. |
16e5a33
to
43ff0c6
Compare
- Add TLS Support by accepting TLS options and passing them to Mojo::IOLoop::Client.
43ff0c6
to
a356a19
Compare
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 for the contribution, but I think there's too many attributes. Also, the documentation should get some updates.
@@ -13,3 +13,4 @@ | |||
/META* | |||
/MYMETA* | |||
/pm_to_blib | |||
.idea/ |
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.
What is this?
has 'tls_ca'; | ||
has 'tls_cert'; | ||
has 'tls_key'; | ||
has 'tls_options'; |
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.
I would rather have this part of the "url", like https://docs.mojolicious.org/Mojo/Server/Daemon#listen
Example:
Mojo::URL->new("redis://localhost?ca=ca.pem&cert=cert.pemt&key=key.pm")
And "tls" can be set to true if key and cert exists.