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

Redis Cluster: Cluster ID / Name #6129

Open
wants to merge 1 commit into
base: unstable
Choose a base branch
from
Open

Conversation

dw130
Copy link

@dw130 dw130 commented May 28, 2019

Hi:
I face a problem which desc in #2472

It is truly dangerous when a company own many clusters and all ips are visible,because some somebody else may join many redis cluster together intentionally or unintentionally,It may be
an very serious problem

So I add a cluster-name testaa in redis.conf, hash the cluster name string into a int flag,
check the int flag when counter a CLUSTERMSG_TYPE_MEET  message between cluster nodes.

This problem is desc in #2472 as an proposed-feature feature,
But I really need this feature, could you please take a look at my-code and check as it is right as I am not quite sure. Thanks very much

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wding seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

None yet

2 participants