Merge pull request #1508 from zdreamx/main-2

SliverHorn · web-flow · commit 639b930ac933 · 2023-08-10T16:59:48.000+08:00
bugfix: fixed redis.go Set function
diff --git a/server/utils/captcha/redis.go b/server/utils/captcha/redis.go
@@ -27,11 +27,12 @@ func (rs *RedisStore) UseWithCtx(ctx context.Context) base64Captcha.Store {
 	return rs
 }
 
-func (rs *RedisStore) Set(id string, value string) {
+func (rs *RedisStore) Set(id string, value string) error {
 	err := global.GVA_REDIS.Set(rs.Context, rs.PreKey+id, value, rs.Expiration).Err()
 	if err != nil {
-		global.GVA_LOG.Error("RedisStoreSetError!", zap.Error(err))
+		return err
 	}
+	return nil
 }
 
 func (rs *RedisStore) Get(key string, clear bool) string {

Original file line number	Diff line number	Diff line change
`@@ -27,11 +27,12 @@ func (rs *RedisStore) UseWithCtx(ctx context.Context) base64Captcha.Store {`
`27`	`27`	`return rs`
`28`	`28`	`}`
`29`	`29`
`30`		`-func (rs *RedisStore) Set(id string, value string) {`
	`30`	`+func (rs *RedisStore) Set(id string, value string) error {`
`31`	`31`	`err := global.GVA_REDIS.Set(rs.Context, rs.PreKey+id, value, rs.Expiration).Err()`
`32`	`32`	`if err != nil {`
`33`		`- global.GVA_LOG.Error("RedisStoreSetError!", zap.Error(err))`
	`33`	`+ return err`
`34`	`34`	`}`
	`35`	`+ return nil`
`35`	`36`	`}`
`36`	`37`
`37`	`38`	`func (rs *RedisStore) Get(key string, clear bool) string {`