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/totp-flow #9

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

Conversation

tweinreich
Copy link
Contributor

Hey @tschuehly,

I have another PR for you but I don't expect you to merge it as is because I saw you refactored a bit more in the SNAPSHOT and this is not yet as complete as you'd want it to be (missing exception handling and tests). Still I thought this may be usefull for you if you refactor it a little.

Here is a short overview:

  • invite user by email using the service account, incl. meta data
  • sign up users with email incl. meta data (with or without service account)
  • enroll, unenroll, and verify MFA (only TOTP for now)
  • low-level: runGoTrueWithResult() to be able to pass a result (e.g. I use your starter in a Spring Boot app and there is further business logic which needs the results of the service)

Have a great start into the week and keep up the great work :)

@tschuehly
Copy link
Owner

Very nice! Will take a look next weekend or so!

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