Skip to content

fix(auth): prevent exchange rate expired due to missing user flags#660

Merged
bmc08gt merged 1 commit intocode/cashfrom
fix/exchange-rate-expired-missing-flags
May 4, 2026
Merged

fix(auth): prevent exchange rate expired due to missing user flags#660
bmc08gt merged 1 commit intocode/cashfrom
fix/exchange-rate-expired-missing-flags

Conversation

@bmc08gt
Copy link
Copy Markdown
Collaborator

@bmc08gt bmc08gt commented May 2, 2026

  • Move Events.OnLoggedIn from set(flags) to set(authState) with transition guard so it fires exactly once per session
  • Retry getUserFlags() with retryable(maxRetries=3) on login
  • Relax updateUserFlags() guard to isAtLeastRegistered and promote auth state on successful recovery
  • Observe network restoration to retry flags fetch when stuck in Registered state
  • Add real time-based expiration to exchangeDataFor() with 15m default fallback instead of returning null when timeout is unset

@bmc08gt bmc08gt self-assigned this May 2, 2026
- Move Events.OnLoggedIn from set(flags) to set(authState) with
  transition guard so it fires exactly once per session
- Retry getUserFlags() with retryable(maxRetries=3) on login
- Relax updateUserFlags() guard to isAtLeastRegistered and promote
  auth state on successful recovery
- Observe network restoration to retry flags fetch when stuck in
  Registered state
- Add real time-based expiration to exchangeDataFor() with 15m default
  fallback instead of returning null when timeout is unset

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt force-pushed the fix/exchange-rate-expired-missing-flags branch from b469f54 to e366322 Compare May 2, 2026 20:14
@bmc08gt bmc08gt added type: fix Bug fix area: auth Login, session, access keys, identity next release Slotted for inclusion in next release labels May 3, 2026
@bmc08gt bmc08gt merged commit 8708564 into code/cash May 4, 2026
1 check passed
@bmc08gt bmc08gt deleted the fix/exchange-rate-expired-missing-flags branch May 4, 2026 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: auth Login, session, access keys, identity next release Slotted for inclusion in next release type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant