handle error before populating cache

5be2332c · Marcus Efraimsson · 6ef941ea · 5be2332c
Commit 5be2332c authored Nov 05, 2018 by Marcus Efraimsson
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletions

pkg/services/sqlstore/user.go
+5 -1

No files found.
--- a/pkg/services/sqlstore/user.go
+++ b/pkg/services/sqlstore/user.go
@@ -353,8 +353,12 @@ func (ss *SqlStore) GetSignedInUserWithCache(query *m.GetSignedInUserQuery) erro
 	}

 	err := GetSignedInUser(query)
-	ss.CacheService.Set(cacheKey, query.Result, time.Second*5)
+	if err != nil {
 		return err
+	}
+
+	ss.CacheService.Set(cacheKey, query.Result, time.Second*5)
+	return nil
 }

 func GetSignedInUser(query *m.GetSignedInUserQuery) error {