-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
stages/authenticator_email: Email OTP #12630
base: main
Are you sure you want to change the base?
stages/authenticator_email: Email OTP #12630
Conversation
✅ Deploy Preview for authentik-docs canceled.
|
✅ Deploy Preview for authentik-storybook canceled.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #12630 +/- ##
==========================================
+ Coverage 92.75% 92.79% +0.03%
==========================================
Files 785 792 +7
Lines 39623 40109 +486
==========================================
+ Hits 36754 37218 +464
- Misses 2869 2891 +22
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…e AuthenticatorEmailStageForm.ts
@rissson I pushed the generated |
Yes, that's the way to go about it |
…henticatorEmailStage in tasks.py
…henticator Email stage UI
…ts/tests/test_models.py::TestModels, add tests for the new __str__ method
Signed-off-by: Marcelo Elizeche Landó <[email protected]>
…atorEmailStage, add test for email tasks
…che/authentik into feature/stage_authenticator_email
… if use global settings is on
VideosSet up Demo user without email on their profile Demo user with email on their profile |
Signed-off-by: Jens L. <[email protected]>
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.
Some minor last changes, otherwise LGTM
def __str__(self): | ||
if not self.pk: | ||
return "New Email Device" | ||
return f"Email Device for {self.user_id}" |
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.
@BeryJu I had to undo your change, self.user_pk
doesn't exists here, I changed to self.user_id
Details
stages/authenticator_email: Add Email Authenticator Stage
Introduces a new authentication stage that enables email-based two-factor authentication. This stage:
closes #3291
Documentation PR: #12853
Checklist
ak test authentik/
)make lint-fix
)If an API change has been made
make gen-build
)If changes to the frontend have been made
make web
)If applicable
make website
)