-
Notifications
You must be signed in to change notification settings - Fork 356
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: add MFA for WebAuthn #1775
base: master
Are you sure you want to change the base?
Conversation
example.env
Outdated
@@ -233,3 +233,6 @@ GOTRUE_HOOK_CUSTOM_SMS_PROVIDER_SECRET="" | |||
# Test OTP Config |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update openapi
@@ -0,0 +1,2 @@ | |||
alter table {{ index .Options "Namespace" }}.mfa_factors add column if not exists web_authn_credential jsonb null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reinstate aaguid
} | ||
for _, factor := range user.Factors { | ||
switch { | ||
case factor.FriendlyName == params.FriendlyName: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Validation checks can be condensed into a single function. Work for that is done here: #1780
Not quite ready yet, please check back after 12pm CEST tomorrow.. |
Pull Request Test Coverage Report for Build 11113961168Details
💛 - Coveralls |
What kind of change does this PR introduce?
Adds MFA for WebAuthn
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.