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

feat: cleanup mfa logic in signer #1409

Merged
merged 1 commit into from
Mar 4, 2025
Merged

Conversation

blakecduncan
Copy link
Collaborator

@blakecduncan blakecduncan commented Mar 4, 2025

Pull Request Checklist


PR-Codex overview

This PR focuses on enhancing multi-factor authentication (MFA) handling in the codebase, introducing new error classes and functions to manage MFA requirements more effectively.

Detailed summary

  • Added MfaRequiredError class to handle MFA requirements.
  • Introduced parseMfaError function to parse MFA errors.
  • Updated various files to replace multiFactor with multiFactors.
  • Refactored authentication methods to support multiple MFA factors.
  • Improved error handling in email authentication flow.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Mar 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 4, 2025 7:01pm
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 4, 2025 7:01pm

@blakecduncan blakecduncan merged commit bde11ff into mfa/demo Mar 4, 2025
7 checks passed
@blakecduncan blakecduncan deleted the blake/signer-mfa-clean-up branch March 4, 2025 19:31
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