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

Add expire and refresh feature for tokens #20

Closed
szerhusenBC opened this issue Jan 31, 2017 · 8 comments
Closed

Add expire and refresh feature for tokens #20

szerhusenBC opened this issue Jan 31, 2017 · 8 comments
Milestone

Comments

@szerhusenBC
Copy link
Owner

There should be a possibility to give a tokens a specific validity duration so that tokens expire after a certain interval. Furthermore there should be a possibility in the JS client to refresh the token. Maybe there is a kind of countdown in the UI, so that a user can see if a token expires.

  • Check expiration in org.zerhusen.security.JwtAuthenticationTokenFilter#doFilterInternal
  • Token refresh under org.zerhusen.security.controller.AuthenticationRestController#refreshAndGetAuthenticationToken
@houssemlou
Copy link

how can we refresh an expired token , because while i try to get claims from the expired token i get an ExpiredJwtException

@szerhusenBC
Copy link
Owner Author

It gave an answer here:

#70 (comment)

@szerhusenBC
Copy link
Owner Author

Closed, because I published a complete new version.

@jmdopereiro
Copy link

jmdopereiro commented Mar 9, 2020

Hi Stephan, thank you for the demo!

I assume that in the new version you have removed the refresh token feature right ?

I guess is because it could be a vulnerability ? What do you think about Blacklisting tokens ?

Please let me know if I should open another issue for my questions.

@szerhusenBC
Copy link
Owner Author

Hi @jmdopereiro ,

I didn't implement the refresh endpoint so far, because I didn't find the time for that, yet ;)

There was already a question about blacklisting / invalidating tokens:

#102

@jmdopereiro
Copy link

Hi Stephan, is it ok for you if I create a branch with the refresh endpoint ?

Thanks

Jose

@szerhusenBC
Copy link
Owner Author

szerhusenBC commented Mar 10, 2020

@jmdopereiro yeah, do it! I created a new issue for that #109

@jmdopereiro
Copy link

Hi Stephan, sorry for the delay, this crisis made me to switch gears.

I saw the ticket, I will continue commenting on it.

Thanks!

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

3 participants