Skip to content

Count every revoked session in revokeOtherSessions - #786

Draft
dowski wants to merge 1 commit into
dowski/expired-session-cleanupfrom
dowski/revoke-count-live-sessions
Draft

dowski wants to merge 1 commit into
dowski/expired-session-cleanupfrom
dowski/revoke-count-live-sessions

Conversation

@dowski

@dowski dowski commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

The cleanup loop now erases expired sessions, so revokeOtherSessions no
longer filters them out of its count with the app's TTL: it returns how
many sessions it revoked. A session that expired moments ago, before the
loop reached it, can now be revoked and counted too.

That was the last use of the TTL defaults on the app side, so lib/ttl.ts
goes away and the defaults move into component/expiry.ts.

Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com

@vercel

vercel Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
auth-v2 Ready Ready Preview Oct 27, 2026 3:06pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
convex-auth-docs Ignored Ignored Oct 27, 2026 3:06pm UTC

Request Review

dowski commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator Author

@pkg-pr-new

pkg-pr-new Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
npm i https://pkg.pr.new/@convex-dev/auth@786

commit: e77dca0

The cleanup loop now erases expired sessions, so revokeOtherSessions no
longer filters them out of its count with the app's TTL: it returns how
many sessions it revoked. A session that expired moments ago, before the
loop reached it, can now be revoked and counted too.

That was the last use of the TTL defaults on the app side, so lib/ttl.ts
goes away and the defaults move into component/expiry.ts.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview – auth-v2 — e77dca02 Deployed Oct 27, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant