peter/ekster
1
0
Fork 0
Code Issues 3 Pull Requests Projects 3 Releases 1 Wiki Activity

Cleanup error handling when setting token

Browse Source
This commit is contained in:
Peter Stuifzand 2018-09-15 16:13:08 +02:00
parent 76bad074c7
commit 8792285eb4
Signed by: peter
GPG Key ID: 374322D56E5209E8
1 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
cmd/eksterd/auth.go
View File

@ -63,21 +63,12 @@ func (b *memoryBackend) cachedCheckAuthToken(conn redis.Conn, header string, r *
}
// setCachedTokenResponseValue remembers the value of the auth token response in redis
func setCachedTokenResponseValue(conn redis.Conn, key string, r *auth.TokenResponse) {
func setCachedTokenResponseValue(conn redis.Conn, key string, r *auth.TokenResponse) error {
_, err := conn.Do("HMSET", redis.Args{}.Add(key).AddFlat(r)...)
if err != nil {
log.Printf("Error while setting token: %v\n", err)
} else {
_, err = conn.Do("EXPIRE", key, uint64(10*time.Minute/time.Second))
if err != nil {
log.Printf("Error while setting expire on token: %v\n", err)
log.Println("Deleting token")
_, err = conn.Do("DEL", key)
if err != nil {
log.Printf("Deleting token failed: %v", err)
}
}
return fmt.Errorf("error while setting token: %v", err)
}
conn.Do("EXPIRE", key, uint64(10*time.Minute/time.Second))
}
// getCachedValue gets the cached value from Redis