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

Some reform about this sso system #33

Open
gyf6067 opened this issue Mar 7, 2019 · 1 comment
Open

Some reform about this sso system #33

gyf6067 opened this issue Mar 7, 2019 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@gyf6067
Copy link

gyf6067 commented Mar 7, 2019

  1. 对调 userID 与 version,否则同一用户在redis中仅存在一份session信息;

  2. 在已有过滤器[XxlSsoWebFilter]基础上,增加过滤器:仅校验sessionID的有效性,失败时不跳转SSO登陆页;

  3. XxlSsoUser中的[plugininfo]改为:Map<String, Object> ;

  4. 修复bug:SsoTokenLoginHelper.loginCheck():用毫秒比分钟;

  5. 修复bug:SsoLoginStore.put(): redisExpireMinite 需改为 xxlUser.getExpireMinite() ;

@xuxueli xuxueli added bug Something isn't working help wanted Extra attention is needed and removed bug Something isn't working labels Dec 13, 2019
@xuxueli
Copy link
Owner

xuxueli commented Dec 13, 2019

如下Copy From:#29

貌似有一个12小时后刷新redis的bug

SsoTokenLoginHelper.java 的第74行:
// After the expiration time has passed half, Auto refresh
if ((System.currentTimeMillis() - xxlUser.getExpireFreshTime()) > xxlUser.getExpireMinite()/2) {
xxlUser.setExpireFreshTime(System.currentTimeMillis());
SsoLoginStore.put(storeKey, xxlUser);
}
不能用毫秒与分钟进行比较。。。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants