authgear / authgear-server

Open source alternative to Auth0 / Firebase Auth
https://www.authgear.com
Apache License 2.0
73 stars 30 forks source link

Change the names of the Password Strength Level #4225

Open fungc-io opened 1 month ago

fungc-io commented 1 month ago

Our password strength level is conflicting with the description of the strength estimation library.

Suggest to change them to:

It's also easier to understand for layman

image.png

linear[bot] commented 1 month ago

DEV-1310 Change the names of the Password Strength Level

chpapa commented 1 month ago

Actually the name is not very understandable anyway.

What about if we rename it like these for our Authgear Portal?

Or something like that? to indicate both "no of guesses" and what type of attacks to expect protection from.

fungc-io commented 1 month ago

Yea, it will be useful to have a description. But it maybe too wordy in a dropdown

I'm thinking of presenting them with a vertical slider like this:

image.png
fungc-io commented 1 month ago

Hi @arvinho,

This refers to the password strength here,

We are thinking to 1) change the naming, 2) add description to each option.

What to you think is a good way to present them?

image.png
arvinhoux commented 1 month ago

@fungc-io Here are suggestions for adjusting the names of password strength levels.

For the descriptions, if they are aimed at general users, I think expressing them in this way may resonate better with users. Providing examples would enhance understanding. Of course, these examples can be modified according to our actual password setting rules.

chpapa commented 1 month ago

@arvinhoux

  1. It's not "no password set", password is still required (default is min length = 8)
  2. The description, while I agree is easier for people to understand, are not technically accurate. I'm afraid it would also create trouble for us when enterprise users review it.

2nd thoughts, we should actually mention the password strength level here is based on zxcvbn, and maybe have a link to our help doc to explain it better.

Maybe we just add on the UI it is based on zxcvbn, and use the official description and score to make it clearer?

arvinhoux commented 1 month ago

I misunderstood earlier. Adjusting it to "None" to represent not using it may be more direct.

Alternatively, we can consider including a reference table in our documentation mapping to the official zxcvbn explanations, and provide a link to the official source. The benefit of doing so is that it can aid general users' understanding while also avoiding potential issues arising from misunderstandings.

fungc-io commented 1 month ago

@arvinho, such table and details are already in the help doc: https://docs.authgear.com/security/password-strength#how-password-strength-is-calculated-in-authgear

Maybe just add a paragraph/help text in the minimum strength selection in the Portal UI and link to the doc.

"Strength level based on [zxcvbn](link) algorithm"

Can you help design that?