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
Retain 2FA authorization #15749
Retain 2FA authorization #15749
Conversation
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.
LGTM
@victorphoenix3 is it ready ?
I will test it soon
Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes Yes, the PR seems to fix the bug ,do test it. |
My guess is that the 2FA settings are not loaded from the database |
@williamdes oh, I'll try to debug and pin down the cause. |
I am also trying to debug actually ^^ This first one who finds posts it here :) |
Done, I used your fix because it was the best option |
Signed-off-by: Jayati Shrivastava <gaurijove@gmail.com>
Codecov Report
@@ Coverage Diff @@
## QA_5_0 #15749 +/- ##
============================================
- Coverage 53.05% 52.75% -0.31%
- Complexity 14153 14825 +672
============================================
Files 482 473 -9
Lines 62127 60999 -1128
============================================
- Hits 32964 32178 -786
+ Misses 29163 28821 -342 |
@victorphoenix3 can you review so I can merge quickly ? |
Thank you for your contribution @victorphoenix3 ! |
Fixes #15351 .
Signed-off-by: Jayati Shrivastava gaurijove@gmail.com
BEFORE
AFTER
Before submitting pull request, please review the following checklist:
Signed-off-by
line as described in our DCO. This ensures that the work you're submitting is your own creation.