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.

Sign up for GitHub

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

Jump to bottom

JwtFilter 中的 refreshToken 方法含有 TODO 标记 #9

Closed
chunsiyang opened this issue Nov 29, 2018 · 6 comments
Closed

JwtFilter 中的 refreshToken 方法含有 TODO 标记 #9

chunsiyang opened this issue Nov 29, 2018 · 6 comments

Comments

@chunsiyang
Copy link
Contributor

该方法是还没有完成么，还是完成后没有去除 TODO 标记
@dolyw
Copy link
Owner

dolyw commented Nov 29, 2018

完成后没有去除 TODO 标记

@dolyw
Copy link
Owner

dolyw commented Nov 29, 2018

你fork去除提交下吧

@chunsiyang
Copy link
Contributor Author

你叉去除提交下吧

在做了，有个疑问想请教下，refreshToken 方法中现在只实现了对 未过期AccessToken 的刷新处理，那么AccessToken过期但是RefreshToken未过期的刷新操作是在哪里实现的呢，谢谢啦

@dolyw
Copy link
Owner

dolyw commented Nov 29, 2018

都是在同一个地方实现的呀
image
认证时捕获到异常是AccessToken过期的异常也进行刷新

@dolyw
Copy link
Owner

dolyw commented Nov 29, 2018

image

@chunsiyang
Copy link
Contributor Author

好的，谢谢了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chunsiyang @dolyw