Conversation
|
Something went wrong with the PR, you have commits in there which are not yours. Could you try rebasing on master and check that your branch only contains your changes? |
d5204ad to
082d428
Compare
Signed-off-by: navid <[email protected]>
|
tests need adjustments |
Signed-off-by: MohammadReza vahedi <[email protected]>
Update AdminTest.php
Signed-off-by: Michiel de Jong <[email protected]>
|
@mrvahedi68 some tests are still failing |
|
@blizzz Now all tests passed except one integration test that has the same error on other pull requests on the master branch. Can review PR again? |
|
Bump |
|
I was OOO last weeks. Thank for your fixing the tests. The failing master integration should be unrelated, I like to doublecheck this anyway. Please stay tuned. Just one question: Would the IdP actually send a successful authentication response despite failing/missing MFA? |
Yes, authentication can be
|
|
The failure is due to an incompatible sqlite3 version. The very base image of the container is centos7, where only sqlite 3.7 is available. We have to solve this, but it shall not block this PR. |
|
Thanks! |
Adding MFA verification mapping configuration in the admin panel.
Adding the MFA verified flag Into the formatted user data in UserBackend.