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

Improve password security - length and characters #3126

Closed
jrtricafort opened this issue Jun 29, 2018 · 5 comments
Closed

Improve password security - length and characters #3126

jrtricafort opened this issue Jun 29, 2018 · 5 comments

Comments

@jrtricafort
Copy link
Contributor

jrtricafort commented Jun 29, 2018

Describe the solution you'd like

  • Increase minimum length of passwords.
  • Make sure we are allowing all characters that should be valid.
  • Verify we allow and can store unicode characters - if we can't store them, don't allow them
  • Notify users that they'll need to change their password once this runs
    [more?]

Aha! Link: https://ushahiditeam.aha.io/features/PROD-320

@rjmackay
Copy link
Contributor

rjmackay commented Jul 1, 2018

Added:
Verify we allow and can store unicode characters - if we can't store them, don't allow them

@tuxpiper
Copy link
Member

tuxpiper commented Jul 1, 2019

Notify users that they'll need to change their password once this runs

I'd say this is a side task that concerns mainly our SaaS and may be of lower priority to the rest.

I'd suggest to drop it from the description and go ahead with this issue in an upcoming release.

@tuxpiper
Copy link
Member

tuxpiper commented Jul 1, 2019

Side-note: initially I thought that there should probably be additional requirements such as use of digits, symbols, mixed-case ... but then I realized we probably not have yet a very good UI to communicate such sophistication of requirements, and hence lots of users may end up being annoyed.

@Erioldoesdesign
Copy link

@tuxpiper We for sure don't have intelligent UI that updates a password criteria list with nice green ticks as most password thingy do nowadays but we can certainly add in helper text that suggest a password contain 1 capital letter, symbol and numbers.

@rowasc rowasc added this to the Parking Lot milestone Sep 4, 2020
@CeciliaHinga
Copy link
Contributor

@tuxpiper I guess this was already discussed

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

7 participants