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

Improve /logout speed #47

Closed
thorwebdev opened this issue Nov 25, 2020 · 2 comments
Closed

Improve /logout speed #47

thorwebdev opened this issue Nov 25, 2020 · 2 comments
Labels
good first issue Good for newcomers perf performance

Comments

@thorwebdev
Copy link
Member

Chore

Describe the chore

The supabase.auth.logout() function takes consistently more than 2 seconds to resolve. Is there anything we can do to speed this up? It's a bit awkward for that much delay after hitting the signout button.

Additional context

Tested here: https://nextjs-subscription-payments-starter.vercel.app/
image

@icecream78
Copy link
Contributor

icecream78 commented Nov 25, 2020

Somewhy mysql doesn't use index while logout.
image
I'll try figure out how to tune perfomance of this query, but I'm not PRO in stuff like this and will take some time=)

@J0 J0 added the perf performance label Jan 19, 2022
@J0 J0 added the good first issue Good for newcomers label May 9, 2022
@J0
Copy link
Contributor

J0 commented Aug 29, 2022

Should be resolved by the move to sessions and the change in the underlying query to make use of the sessions model rather than refresh tokens. Going to close for now but feel free to re-open if issue persists.

cc: @kangmingtay @hf

Realized after posting that a tag wasn't quite needed(bc there's no follow up action required) -- please ignore

@J0 J0 closed this as completed Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers perf performance
Projects
None yet
Development

No branches or pull requests

3 participants