-
Notifications
You must be signed in to change notification settings - Fork 310
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
Fix refresh cookie token bugs #227
Conversation
Thanks for submitting this pull request. I'm setting up Circle CI to build this, but I noticed the test Can you have a look? You can run tests locally using |
Sure, I will check this. |
@iMerica all tests passed. |
LGTM 👍 |
There was a bug in which
/token/refresh/
did not read and update the refresh token cookie. A small refactoring is also included.