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

It adds new refresh token when another token already exist for logged in user #274

Open
mdmuhtasimfuadfahim opened this issue Oct 25, 2023 · 2 comments
Open

It adds new refresh token when another token already exist for logged in user #274

mdmuhtasimfuadfahim opened this issue Oct 25, 2023 · 2 comments

Comments

@mdmuhtasimfuadfahim
Copy link

I have hit the login API 10-15 times. All the times it creates new refresh token under tokens database. This bug should be solved as well as the #267 issue.
@mdmuhtasimfuadfahim mdmuhtasimfuadfahim changed the title It adds new refresh token when another token is already exist under that user It adds new refresh token when another token already exist for logged in user Oct 25, 2023
mdmuhtasimfuadfahim added a commit to mdmuhtasimfuadfahim/node-express-boilerplate that referenced this issue Oct 25, 2023
@mdmuhtasimfuadfahim
Fixed: Multiple tokens creation for already logged in users
869a47e 
Issue hagopj13#274 resolved
@mdmuhtasimfuadfahim mdmuhtasimfuadfahim mentioned this issue Oct 25, 2023
Open
@chintanshahts
Copy link

chintanshahts commented Oct 28, 2023 •
edited
Loading

It depends on your requirement. User can log in from multiple devices.
If your product does not allow the multiple device login you need to remove the refresh token else all good.

@mdmuhtasimfuadfahim
Copy link
Author

mdmuhtasimfuadfahim commented Oct 28, 2023 •
edited
Loading

@chintanshahts yes. But with this boilerplate there is no device information is going at the time of login. So I think this would be a good solution and please check the issue #267. To solve this issue this might help.

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
@mdmuhtasimfuadfahim @chintanshahts