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

Increase Password Max Length #54

Closed
1 of 3 tasks
gyhughes opened this issue Oct 13, 2022 · 2 comments
Closed
1 of 3 tasks

Increase Password Max Length #54

gyhughes opened this issue Oct 13, 2022 · 2 comments

Comments

@gyhughes
Copy link

gyhughes commented Oct 13, 2022

What is the problem you are trying to solve with this feature?

When registering a new user, the Register button is disabled if the password is larger than 19 characters.
This character limit is also present in the "Change Password" modal.

What is the solution you are proposing?

Increase the password max length to 20+ characters.

What alternatives have you considered?

Using a less complex/secure password.

Additional context

Screenshot

Contributions

  • I have searched through existing issues and feature requests to see if my idea has already been proposed.
  • If this feature is accepted, I would be willing to help implement and maintain this feature.
  • If this feature is accepted, I'm willing to sponsor the development of this feature.
@gyhughes gyhughes changed the title Increase Password Length Increase Password Max Length Oct 13, 2022
@hay-kot
Copy link
Owner

hay-kot commented Oct 13, 2022

There isn't a maximum length for the password. The issue you're likely running into is that your password contains one of the keywords we flag for security reasons

const flaggedWords = ["password", "homebox", "admin", "qwerty", "login"];

If you're using any of these words in your password it will automatically fail the complexity requirement.

CleanShot 2022-10-13 at 08 31 37

@gyhughes
Copy link
Author

Ah you are right. I had "homebox" at the end of the password. Can either close this or change this to adding validation message about flagged words.

@hay-kot hay-kot closed this as completed Oct 14, 2022
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

No branches or pull requests

2 participants