Skip to content

fix(ios): clear biometric Keychain token on logout#32149

Merged
Jasonnnz merged 2 commits into
Jasonnnz/ios-session-persistfrom
ios-session-persist/pr-3-logout-cleanup
May 26, 2026
Merged

fix(ios): clear biometric Keychain token on logout#32149
Jasonnnz merged 2 commits into
Jasonnnz/ios-session-persistfrom
ios-session-persist/pr-3-logout-cleanup

Conversation

@Jasonnnz
Copy link
Copy Markdown
Contributor

Summary

  • Call deleteBiometricToken() in auth store logout to clear stale Keychain entries
  • Fire-and-forget — never blocks or throws on cleanup failure
  • Prevents stale biometric token from restoring a session after explicit logout

Part of plan: ios-session-persist.md (PR 3 of 4)

Vellum Assistant and others added 2 commits May 26, 2026 14:58
@Jasonnnz Jasonnnz merged commit 14e08c1 into Jasonnnz/ios-session-persist May 26, 2026
1 of 4 checks passed
@Jasonnnz Jasonnnz deleted the ios-session-persist/pr-3-logout-cleanup branch May 26, 2026 19:01
Jasonnnz added a commit that referenced this pull request May 26, 2026
…metric recovery (#32153)

* fix(ios): enable CapacitorCookies to persist session cookies across app kills (#32147)

Co-authored-by: Vellum Assistant <assistant@vellum.ai>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(ios): add WKAppBoundDomains for reliable cookie persistence (#32148)

Co-authored-by: Vellum Assistant <assistant@vellum.ai>

* fix(ios): recover session from biometric Keychain when cookies are missing (#32150)

Co-authored-by: Vellum Assistant <assistant@vellum.ai>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(ios): clear biometric Keychain token on logout (#32149)

Co-authored-by: Vellum Assistant <assistant@vellum.ai>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: remove stray merge conflict marker from auth-store test

---------

Co-authored-by: Vellum Assistant <assistant@vellum.ai>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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