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

SSL connections from peers with matching wildcard name are always rejected [JIRA: RIAK-1506] #698

Closed
Basho-JIRA opened this issue Feb 9, 2015 · 4 comments

Comments

@Basho-JIRA
Copy link

At [https://github.com/basho/riak_core/blob/develop/src/riak_core_ssl_util.erl#L189] we perform an exact string comparison on an X.509 certificate's CN and reject all connection attempts if the peer certificate's name matches our own.

The problem arrises when the CN begins with a wildcard, which is not an uncommon practice where pools of hosts are involved. In this case, there are legitimate use cases where the connection's peers may present the same wildcarded certificate, which we reject - even if there's an explicit ACL allowing them.

The code is relatively easy to fix, once we decide how we want to fix it. The easiest path is to make it through the initial test if the CN is a wildcard, allowing the ACL test(s) to be applied. We could also add a configuration flag to allow this specific use case.

Initiated by customer issue [https://basho.zendesk.com/agent/tickets/10030]

[Created in JIRA by Ted Burghart]

@Basho-JIRA
Copy link
Author

If it would be helpful, it sounds as though the customer (see: https://basho.zendesk.com/agent/tickets/10030) would be more than happy to test a patch to verify that the fix resolves their issue.

_[posted via JIRA by Justin Pease]_

@Basho-JIRA
Copy link
Author

Also, worth noting: Customer expectation is that this will be in 2.0.6 (via Nate Gustafson).

_[posted via JIRA by Justin Pease]_

@Basho-JIRA
Copy link
Author

The work for this is complete but is waiting on 2.0.5 release to be merged.

riak_core PR: #701
riak_test PR: basho/riak_test#737

_[posted via JIRA by Douglas Rohrer]_

@seancribbs
Copy link
Contributor

Addressed by #701

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

No branches or pull requests

2 participants