-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
📧 feat: email verification #2344
Conversation
a01fd36
to
d0fe4a7
Compare
@berry-13 Thank you for your contribution! I merged this PR. After registration, the backend often shows But even if I receive the verification email, when I click the link "http://localhost:3080/verify?token=xxx&userId=xxx", it shows "Unexpected Application Error! |
…ly when ALLOW_UNVERIFIED_EMAIL_LOGIN = true
bc21421
to
f10833c
Compare
some technical debt with this PR: unverified users can potentially remain unverified forever. the solution would be to utilize |
…eral fixes from recent changes
…$set and $unset operations, createUser now returns Id only
…heck, move saveBuffer init to buffer condition
… link to Login Form, make registration redirect shorter
9d67f30
to
5acc0a5
Compare
Summary
added email verification for secure sign-ups 💪
(forgot to change the branch name oops)
Change Type
Testing
Email tested with
gmail
SMTPChecklist