astelm/minio
1
0
Fork 0
mirror of https://github.com/pgsty/minio.git synced 2026-08-02 08:35:58 +03:00
Code Issues Packages Projects Releases Wiki Activity

Implement CLIENT SETNAME for Redis connections (#9876)

Browse Source 
Add note about CLIENT SETNAME needing auth
This commit is contained in:
Michael Mayr
2020-06-19 22:28:28 +02:00
committed by Harshavardhana
parent 0513ac72ff
commit 98ac2c452a
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/event/target/redis.go
+9 -3
View File
@@ -267,11 +267,17 @@ func NewRedisTarget(id string, args RedisArgs, doneCh <-chan struct{}, loggerOnc
return nil, err return nil, err
} }
if args.Password == "" { if args.Password != "" {
return conn, nil if _, err = conn.Do("AUTH", args.Password); err != nil {
cErr := conn.Close()
targetID := event.TargetID{ID: id, Name: "redis"}
loggerOnce(context.Background(), cErr, targetID)
return nil, err
}
} }
if _, err = conn.Do("AUTH", args.Password); err != nil { // Must be done after AUTH
if _, err = conn.Do("CLIENT", "SETNAME", "MinIO"); err != nil {
cErr := conn.Close() cErr := conn.Close()
targetID := event.TargetID{ID: id, Name: "redis"} targetID := event.TargetID{ID: id, Name: "redis"}
loggerOnce(context.Background(), cErr, targetID) loggerOnce(context.Background(), cErr, targetID)