Skip to content
This repository has been archived by the owner on Feb 21, 2023. It is now read-only.

Correct error message on sentinel discovery of master/slave with password. #284

Conversation

azinoviev
Copy link
Contributor

This will help to get ReplyError "Master X error: ERR invalid password"/"Master X error: NOAUTH Authentication required." instead of MasterNotFoundError "No master found for X" when discovering password protected master/slave via Sentinel without password (or with wrong password) provided.

Pyenv's .python-version config added to .gitignore.

@azinoviev azinoviev force-pushed the feature/sentinel-passwd-fix branch from 7be374c to 599bbab Compare August 14, 2017 08:51
@codecov
Copy link

codecov bot commented Aug 14, 2017

Codecov Report

Merging #284 into master will increase coverage by 0.02%.
The diff coverage is 88.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
+ Coverage   96.83%   96.85%   +0.02%     
==========================================
  Files          59       59              
  Lines        7553     7577      +24     
  Branches      599      603       +4     
==========================================
+ Hits         7314     7339      +25     
- Misses        174      175       +1     
+ Partials       65       63       -2
Impacted Files Coverage Δ
tests/sentinel_commands_test.py 100% <100%> (ø) ⬆️
aioredis/sentinel/pool.py 86.71% <100%> (ø) ⬆️
tests/conftest.py 89.21% <66.66%> (-0.76%) ⬇️
aioredis/commands/server.py 85.23% <0%> (+1.34%) ⬆️
tests/sentinel_failover_test.py 88.72% <0%> (+2.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 059c74f...117bbbe. Read the comment docs.

…word.

Pyenv's .python-version config added to .gitignore.
@azinoviev azinoviev force-pushed the feature/sentinel-passwd-fix branch from 599bbab to 117bbbe Compare October 6, 2017 16:56
@popravich popravich added this to the v1.0 milestone Nov 9, 2017
@popravich
Copy link
Contributor

Closing this in favor of #327 (rebased and updated)

@popravich popravich closed this Nov 14, 2017
popravich added a commit that referenced this pull request Nov 14, 2017
#284 Correct error message on sentinel discovery of master/slave with password
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants