-
-
Notifications
You must be signed in to change notification settings - Fork 162
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
Email verification should be there #3317
Comments
@DonnieBLT what's your opinion |
Sounds good, yes. |
can i work on it? |
Yes type /assign |
/assign |
Hello @yashgoyal0110! You've been assigned to OWASP-BLT/BLT issue #3317. You have 24 hours to complete a pull request. |
and one more thing, OTP verification or just simple valid email check? |
We have email confirmation sent through email - I think it will be good to also add a hidden email field and reject the form if it’s filled out by a bot |
I think we should send an OTP or verification email to continue process, once user sign ups for validation purpose |
We do send a verification email. We should delete these users and activity if they are not verified in 24 hours |
⏰ This issue has been automatically unassigned due to 24 hours of inactivity. |
Can i work on it? |
Hello @akdrin! You've been assigned to OWASP-BLT/BLT issue #3317. You have 24 hours to complete a pull request. |
⏰ This issue has been automatically unassigned due to 24 hours of inactivity. |
1 similar comment
⏰ This issue has been automatically unassigned due to 24 hours of inactivity. |
Currently there is no email verification on signup, that's why its also accepting invalid email, there should be some valid checks and even password has some checks (for example it should include uppercase, lowercase, numbers and special characters)
The text was updated successfully, but these errors were encountered: