Skip to content

feat(mfa): add recovery code factor model support - #2728

Open
fadymak wants to merge 1 commit into
fm/auth-1526from
fm/auth-1527
Open

feat(mfa): add recovery code factor model support#2728
fadymak wants to merge 1 commit into
fm/auth-1526from
fm/auth-1527

Conversation

@fadymak

@fadymak fadymak commented Aug 25, 2026

Copy link
Copy Markdown
Contributor
  • Adds new recovery_code factor type and mfa/recovery_code AMR method
    • Those are wired up through the AAL claim checks
  • Introduces NewRecoveryCodeFactor that creates the factor in a verified state since challenge -> verify does not apply
    • Allows a user to specify a friendly name. Falls back to default of Recovery codes if not specified.

@fadymak
fadymak requested a review from a team as a code owner August 25, 2026 12:41

@annabkr annabkr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 Left two comments but I trust you to make the decision about whether to resolve.

Comment thread internal/models/factor.go
Comment thread internal/models/factor_test.go
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