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

Good approach, but... #2

Open
rohanrkamath opened this issue Apr 13, 2024 · 2 comments
Open

Good approach, but... #2

rohanrkamath opened this issue Apr 13, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@rohanrkamath
Copy link

Hey so I was going through some repos for this purpose, as I am too building one. But the issue is 2fa does involve writing the email and password again, along with the totp. This is were I am stuck too, I am trying to add a cookie with the email details and then using in a validation route. But yea, consider this. Not exactly an issue, but not a very good design choice.

@vinodiOS vinodiOS added the enhancement New feature or request label Apr 13, 2024
@vinodiOS
Copy link
Owner

Hey @rohanrkamath! Thanks for sharing your thoughts.if you've implemented a solution or have any code snippets to share, I'd love to take a look and discuss further. Feel free to open a pull request.

@rohanrkamath
Copy link
Author

Sure thank you. I have been building a project management application, which is far from complete. But I have the authentication system completed. You can have a look - https://github.com/rohanrkamath/PTMS/tree/v4

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

No branches or pull requests

2 participants