Skip to content

fix(auth): clear initial setTimeout in stopAutoRefresh #1993

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
Merged
mandarini merged 1 commit into from
Jan 5, 2026
Merged

fix(auth): clear initial setTimeout in stopAutoRefresh #1993

mandarini merged 1 commit into from
Jan 5, 2026

Conversation

@7ttp
Copy link
Contributor

@7ttp 7ttp commented Dec 26, 2025

the setTimeout in startAutoRefresh was not being tracked or cleared, causing timer leaks in deno tests even after calling stopAutoRefresh.

closes #1992
closes #1683

@7ttp 7ttp requested review from a team as code owners December 26, 2025 20:44
@7ttp 7ttp changed the title fix(auth): clear initial setTimeout in stopAutoRefresh fix(auth-js): clear initial setTimeout in stopAutoRefresh Dec 28, 2025
@mandarini mandarini changed the title fix(auth-js): clear initial setTimeout in stopAutoRefresh fix(auth): clear initial setTimeout in stopAutoRefresh Jan 5, 2026
@mandarini mandarini merged commit 7e49a70 into supabase:master Jan 5, 2026
27 of 28 checks passed
@7ttp 7ttp deleted the fix/deno-timer-leak branch January 5, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mandarini mandarini mandarini approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug deno-timer supabaseClient using setInterval and fails in Deno test

2 participants

@7ttp @mandarini