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

JWT Token decoding and Refresh Token expiration checks #118

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

intellitour
Copy link

@intellitour intellitour commented Feb 4, 2022

This adds safe JWT token decoding and refresh token expiration check, as well as token refreshing failure due to expired refresh token.

This probably fixes #107

@intellitour
Copy link
Author

intellitour commented Feb 4, 2022

Please, @teranetsrl @okrad, provide some feedback on this.

@okrad
Copy link
Collaborator

okrad commented Feb 21, 2022

Hi @intellitour, sorry for the late reply, I'm really busy at work lately...
Thank you for the PR, I was hoping to find the time myself to add JWT decoding, so your help is really appreciated!
It would be great to have some tests, would you mind adding them?

@intellitour
Copy link
Author

@okrad as soon as I can, I will do.

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

Successfully merging this pull request may close these issues.

with custome oarth got oauth2Helper responce got error
2 participants