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

fix(core): avoid logout before initiate totp #4606

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

dolcalmi
Copy link
Collaborator

@dolcalmi dolcalmi commented Oct 5, 2024

Fix https://ui.honeycomb.io/galoy/datasets/galoy-bbw/result/C1KR8g2dQJQ?hideCompare

  • if kratosInitiateTotp fail user should not be logged out
  • avoid race condition when session is logged out before being able to initiate totp registration

@github-actions github-actions bot added the core label Oct 5, 2024
@dolcalmi dolcalmi force-pushed the fix-totp-registration-race-condition branch from 5ed1b90 to 3b20f81 Compare October 5, 2024 01:15
@dolcalmi dolcalmi merged commit 0c3a603 into main Oct 5, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants