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
SSL support #291
SSL support #291
Conversation
MySSL = proplists:get_value(ssl_enabled, TheirCaps, false), | ||
TheirSSL = proplists:get_value(ssl_enabled, MyCaps, false), | ||
case {MySSL, TheirSSL} of | ||
{true, false} -> lager:info("FAILED TO USE SSL A"), |
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.
These lager:info weren't really meant to make it into a PR as they were.
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.
Yes, I forgot to do a cleanup commit.
There are several lager:info/lager:error calls that I was using during dev. These need to be cleaned up. |
lager:info("Peer requested SSL but we don't support it"), | ||
{error, no_ssl}; | ||
{true, true} -> | ||
lager:info("Agreed to use SSL"), |
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.
It would be nice if the info could read: Agreed to use SSL from foo to bar.
(source/sink)
replication2_ssl + replication2 tests pass. |
How does this look? |
output looks good:
replication2 + replication2_ssl pass again. |
This is a port of the SSL implementation from Riak's MDC implementation. The app.config arguments are the same, only now they're under riak_core. SSL is negotiated right after capabilities are exchanged, so minimal information is sent 'in the clear'. If one side requests SSL and the other side does not have it enabled, SSL is not allowed to connect.
Add the riak_kv_yessir_backend.erl storage manager for simulation purposes
No description provided.