redundis icon indicating copy to clipboard operation
redundis copied to clipboard

As Sentinel doesn't support AUTH command (as of Redis 4.0.7), passwor…

Open aminet opened this issue 6 years ago • 0 comments

Hi,

Context: 3 nodes under Debian Strech with backports (redis 4.0.7) with sentinel and redundis and an application which need to connect to a Redis server (a master with slaves in replication mode/auto-failover).

With redis configured without requirepass there is no problem.

But with requirepass set up :

./redundis -m mymaster -p ******************************************** 2018-04-19 12:32:08 [redundis] INFO Listening on '127.0.0.1:6379' 2018-04-19 12:32:08 [redundis] INFO Monitoring master... 2018-04-19 12:32:08 [redundis] ERROR Failed to update master - Failed to reach sentinel - ERR unknown command 'AUTH'

The "missing" AUTH command on sentinel is known: https://github.com/antirez/redis/issues/3279#issuecomment-230485169 , so connecting to sentinel with NOAUTH is actually mandatory.

Sentinel could be fixed with https://github.com/antirez/redis/pull/3329

Until it append, this is a little PR as AUTH must only be used for Master (not on Sentinel) and as a result argument --sentinel-password doesn't make send but --master-password does

aminet avatar Apr 20 '18 09:04 aminet