Skip to content
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

Feature: adds recovery codes #14

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

Feature: adds recovery codes #14

wants to merge 25 commits into from

Conversation

susgo
Copy link

@susgo susgo commented Jul 7, 2020

Now it is ready for merge :-)

  • save recovery codes (default 10) to new database table on enabling two-factor-authentication
  • codes show up only one time to user (I have not tested with the given views because we overwrite them in our project)
  • every code can used exactly one time
  • php unit tests added and included via github actions

Please can you add a version tag for actual master and then after merging.

I look forward to your feedback. :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants