fix: basic bug does not cache info by user name hash

auth/strategies/basic/cached.go

@@ -35,7 +35,7 @@ type cachedBasic struct {
 
 func (c *cachedBasic) authenticate(ctx context.Context, r *http.Request, userName, pass string) (auth.Info, error) { // nolint:lll
 	hash := c.hasher.Hash(userName)
-	v, ok := c.cache.Load(userName)
+	v, ok := c.cache.Load(hash)
 
 	// if info not found invoke user authenticate function
 	if !ok {

auth/strategies/basic/cached_test.go

@@ -88,6 +88,28 @@ func TestNewCached(t *testing.T) {
 	}
 }
 
+func TestCachedUserNameHash(t *testing.T) {
+	count := 0
+	user := "test"
+	cache := libcache.LRU.New(0)
+	opt := SetUserNameHash(crypto.SHA256, []byte("key"))
+	fn := func(ctx context.Context, r *http.Request, userName, password string) (auth.Info, error) {
+		count++
+		return auth.NewDefaultUser(user, "10", nil, nil), nil
+	}
+	basic := NewCached(fn, cache, opt)
+	r, _ := http.NewRequest("GET", "/", nil)
+	r.SetBasicAuth(user, user)
+	for i := 0; i < 10; i++ {
+		info, err := basic.Authenticate(r.Context(), r)
+		assert.NoError(t, err)
+		assert.Equal(t, info.GetUserName(), user)
+		assert.False(t, cache.Contains(user))
+	}
+	assert.Equal(t, 1, cache.Len())
+	assert.Equal(t, 1, count)
+}
+
 func BenchmarkCachedBasic(b *testing.B) {
 	r, _ := http.NewRequest("GET", "/", nil)
 	r.SetBasicAuth("test", "test")