Skip to content

feat(mfa): recovery code storage model - #2753

Open
fadymak wants to merge 1 commit into
fm/auth-1528from
fm/auth-1529
Open

feat(mfa): recovery code storage model#2753
fadymak wants to merge 1 commit into
fm/auth-1528from
fm/auth-1529

Conversation

@fadymak

@fadymak fadymak commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Adds recovery codes model and queries needed to support the feature.

  • Concurrent requests are serialized using FOR UPDATE on the mfa_recovery_code_sets.user_id
  • Tracks failed-verification count and lockout expiry per set to prevent brute force verifications

@fadymak
fadymak requested a review from a team as a code owner August 27, 2026 10:11

@hf hf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have we added all the indexes?

@fadymak

fadymak commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

Thanks!

Have we added all the indexes?

@hf yup, our query patterns should be covered by the indexes added to the initial schema here: #2725

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.

2 participants