consumer-reports-innovation-lab / TheDigitalStandard

The Digital Standard is an ambitious, community-led effort to build a framework to test and rate products and services on the basis of privacy, security, and data practices.
Creative Commons Attribution 4.0 International
128 stars 44 forks source link

Authentication #103

Open TatevSarg opened 6 years ago

TatevSarg commented 6 years ago

Congrats on expanding the authentication section. The new criteria and indicators look great!! A couple of minor suggestions to consider.

  1. Some of the terms need to be explained/defined to clarify the benchmarks.
  1. Clarify the distinction between the following two indicators: “If the product uses a password/passphrase for authentication, it requires that passwords are at least 8 characters long.” and “If the product uses a password/passphrase for authentication, the password/passphrase may be at least 20 characters long.”

  2. Consider adding an indicator or expanding “The product notifies users when account security settings have changed,” to include notice about any unusual account activity and possible unauthorized access to users’ account (e.g., notice about incorrect log-in attempts).

  3. Consider adding an indicator regarding the ability and process to recover locked accounts.

KatieMcInnis commented 5 years ago
  1. Agree that defining the terms would be good. However, I worry that adding these definitions would make this already unwieldy digital standard even more so. Is there a good place to define these terms? Should we have a glossary stored on the github?

  2. How about: "If the product uses a password/passphrase for authentication, the password, at a minimum, be at least 8 characters long" and "If the product uses a password/passphrase for authentication, the product supports longer passwords of 20+ characters." Does that work?

  3. We have been interpreting that indicator as including such actions. For instance, receiving an email to notify you that your password has changed falls underneath this category. Knowing that, do you still recommend we make this indicator more specific?

  4. Interesting! What do you recommend?

Thanks, @TatevSarg !

secretrobotron commented 5 years ago

Is there a good place to define these terms? Should we have a glossary stored on the github?

I think this is a really good idea. We could just start a glossary.md.