Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

自定义实现cache的时候,发现获取access_token判断方式 #529

Closed
lala0257 opened this issue Jan 19, 2022 · 7 comments
Closed

Comments

@lala0257
Copy link

val := ak.cache.Get(accessTokenCacheKey)
if val != nil {
accessToken = val.(string)
return
}
如果val=“”, 那就不是nil

@lala0257 lala0257 added the bug label Jan 19, 2022
@houseme
Copy link
Collaborator

houseme commented Jun 28, 2022

val := ak.cache.Get(accessTokenCacheKey)
if val != nil {
accessToken = val.(string)
return
}
如果val=“”, 那就不是nil

研究一下 go-redis/redis/v8 文档

@johnpoint
Copy link
Contributor

感觉这里的判断改为判断长度不为0会好一点?

@houseme
Copy link
Collaborator

houseme commented Feb 10, 2023

感觉这里的判断改为判断长度不为0会好一点?

需要俩次判断!看下源码go-redis/redis/v8

@popstk
Copy link
Contributor

popstk commented Oct 18, 2023

#721 这里现在改成了判断空字符串,但是没法兼容以前自定义cache,

  • 以前 自定义cache过期要返回nil
  • 现在 自定义cache过期要返回空字符串

@houseme
Copy link
Collaborator

houseme commented Oct 18, 2023

#721 这里现在改成了判断空字符串,但是没法兼容以前自定义cache,

  • 以前 自定义cache过期要返回nil

  • 现在 自定义cache过期要返回空字符串

建议自己实现一个,可以参考 #714 pr

@popstk
Copy link
Contributor

popstk commented Oct 18, 2023

#721 这里现在改成了判断空字符串,但是没法兼容以前自定义cache,

  • 以前 自定义cache过期要返回nil
  • 现在 自定义cache过期要返回空字符串

建议自己实现一个,可以参考 #714 pr

我知道怎么实现一个自己的cache,这是一个小版本,应该保证兼容性(兼容两种返回的情况),否则使用自定义cache的用户从v2.1.5及以前升级上来,会导致产生panic

@houseme
Copy link
Collaborator

houseme commented Oct 18, 2023

#721 这里现在改成了判断空字符串,但是没法兼容以前自定义cache,

  • 以前 自定义cache过期要返回nil
  • 现在 自定义cache过期要返回空字符串

建议自己实现一个,可以参考 #714 pr

我知道怎么实现一个自己的cache,这是一个小版本,应该保证兼容性(兼容两种返回的情况),否则使用自定义cache的用户从v2.1.5及以前升级上来,会导致产生panic

辛苦提pr 处理吧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants