-
Notifications
You must be signed in to change notification settings - Fork 80
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
Not Invalidating token properly #20
Comments
Hi, thanks for reporting this issue. What HTTP code did you get? Sorry, but I can't reproduce this bug. |
Its not an error code. What I meant is, I login a user - a token is generated. I invalidate the user - a token should be blacklisted. I get the user details - still shows data whereas the token was expired earlier and had to be shown an error message. If you still cant get me, try the login, then use the generated token to invalidate. Next, use same token to try get user. It should not give user details but whereas it does. |
@krisanalfa were you able to reproduce? |
Hi @newkillerbeast2017, can u use this postman collection to reproduce the issue? I can't reproduce. |
Yeah I also don't have the ability to expire the token. the test for it is only checking for a response of 200, so fine, but check invalid token doesn't have an issue. Running authentication and then running invalidate doesn't affect the ability to view any protected material. It seems then then expire after a "while" when it complains of missing segments until authenticated again. |
I can't reproduce this |
Hi @rbnzdave it's been a while. Sorry for late reply. How's your issue status? Have you resolved this? Until now I can't reproduce this issue, so I cannot make any fix or help you. |
I checked out the package, made no changes, connected to one of my existing databases and tested the following routes:
api/auth/login
- token got generatedapi/auth/invalidate
- token invalidatedapi/auth/user
- expired token yet gets the data properlyIsn't it supposed to say unauthorized?
The text was updated successfully, but these errors were encountered: