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
Implement Two Factor Authentication #147
Conversation
I thought I could be clever by using a null secret key as an indicator of it being disabled
Codecov Report
@@ Coverage Diff @@
## master #147 +/- ##
==========================================
+ Coverage 56.61% 57.19% +0.58%
==========================================
Files 85 86 +1
Lines 1957 2007 +50
==========================================
+ Hits 1108 1148 +40
- Misses 849 859 +10
Continue to review full report at Codecov.
|
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.
Generally, superb execution on the frontend side, nothing I feel I have to complain about, looking over it. Layout seems fine too.
The fact that the QR code adapts to the theme is awesome, but I fear that might cause it to become unreadable with certain themes, especially lighter ones, doesn't it? Generally not much of a problem, and we still can add a disclaimer later on.
@@ -68,3 +68,6 @@ hcaptcha: | |||
enabled: false | |||
site_key: '' | |||
secret_key: '' | |||
|
|||
# TOTP Drift period in seconds | |||
otp_drift_period: 30 |
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.
change deployed
Won't do another deployment until recovery codes are implemented. |
Closes #84