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

Bug fix: fix NOAUTH error... #171

Merged
merged 3 commits into from
Dec 25, 2015
Merged

Bug fix: fix NOAUTH error... #171

merged 3 commits into from
Dec 25, 2015

Conversation

dowenliu-xyz
Copy link
Contributor

Fix NOAUTH error when connecting to a cluster with password protection.

Before loading partition views, call auth() method if a none-empty password is provided after connected to any server of a cluster.

@mp911de
Copy link
Collaborator

mp911de commented Dec 25, 2015

Thanks for your PR. You were not required to close #170, Github pulls automatically any updates from the source branch into the PR.

@dowenliu-xyz
Copy link
Contributor Author

Ooh, this is my first time PR in Github. Thank you for telling me this.

mp911de added a commit that referenced this pull request Dec 25, 2015
Bugfix: Use authentication in cluster topology refresh #171
@mp911de mp911de merged commit 1c540b0 into redis:master Dec 25, 2015
@mp911de
Copy link
Collaborator

mp911de commented Dec 25, 2015

Merged.

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.

2 participants