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

VALIDATE THE E-MAIL AND ADD METRICS TO MAKE THE PASSWORD STRONGER #1610

Open
sindhuja184 opened this issue Nov 1, 2024 · 2 comments
Open

Comments

@sindhuja184
Copy link

Describe the bug
The e-mail field in the sign-up form is accepting arbitrary e-mails, we need to validate the e-mails.
I would like to impose a regular expression for checking the valid e-mails.

As password plays a vital role to protect one's personal information. There should be set a guidelines lines minimum number of digits etc. In the present model there is only one condition that is the password should be of length 6 characters.

What would I do if I am assigned the task:

  1. Would continue the rule that the length of the password should be 6 or more
  2. Make sure that password contains combination of uppercase letters, lower case letters, digits, special characters.
  3. Make sure that commonly used passwords are not used.
  4. No sequence of characters.(Eg : abcd, 1234)

To Reproduce
Steps to reproduce the behavior:

  1. Go to SignUp
  2. Enter invalid email
  3. Enter a vague password
  4. See error : Click on submit the form is submitted.

Expected behavior
Invalid e-mails and weak passwords should not be accepted by the form

Screenshots
If applicable, add screenshots to help explain your problem.
image

Additional context
Hope you assign me this issue @ANSHIKA-26, I am eager to work on it.

Copy link

github-actions bot commented Nov 1, 2024

Thanks @sindhuja184 for raising this issue! We'll look into it.We hope you have made sure that a similar issue doesnt exist , if it does, kindly ask to be assigned on that issue

@ANSHIKA-26
Copy link
Owner

this issue had been raised previously , can you recheck once if its already working or not ? if not kindly attach a video of the same , since the PR for solving this issue was already merged , if the issue still exists it will be assigned to you , as of now , the issue is not assigned to you so kindly do not work on it , thanks

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

No branches or pull requests

2 participants