-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Update: Authentication Cheat Sheet #1243
Comments
I like this general direction |
I suggest it be located in its own sheet. The primary email on a users account could not be used for authentication. Not to mention that the authentication sheet has more to so with the second part of the authentication. Ex. Secrets A few things I would challenge are:
I general, I think this would be a great addition and something I feel we don't have enough resources available for. |
Step three where the password or MFA check is needed is the most crucial step. I like this direction. Maybe you can add this under re-authentication? |
@jmanico @Zargath thank you both for your feedback. Going to try to merge both of your suggestions here. Please respond to relevant questions?
Note Jim, do you concur? --
Note Edouard, you ok with this particular process when MFA proves identity? --
...instead of forcing user friction by requiring this non-MFA / password-only user to additionally prove identity by showing they have access to the currently-registered email address's inbox Note Jim do you concur with Edouard? --
Edouard wrote:
My take: It depends on the system.
Note Jim / Edouard, can we please write that AFTER this cheat sheet creation / update is completed? |
I think a relatively small update to the current authn cheat sheet will be enough!
|
@Zargath you ok with that? |
Yes, I'm good with that. |
PR merged, so closing this Issue |
Add "Change Registered Email Address" section to Authentication Cheat Sheet
I looked around and could not find any guidance on how to safely have a user change their registered email address in a system. At first glance, the Authentication Cheat Sheet seems to be the proper place to do this.
If it's in another cheat sheet, please, somebody tell me where to go look?
If another cheat sheet would be a better place to put this guidance, please somebody propose a more relevant cheat sheet?
Add section: "Change Registered Email Address"
...perhaps as a subsection of "User IDs"?
In that section
Here's my suggested process:
Confirm user's authentication cookie / token is still valid
Note to the user that -- if their the originally-registered email address is no longer valid or inaccessible then the user needs to contact Customer Support at to confirm their identity in another way
...and that Customer Support will need to be regularly trained on social engineering attacks...or accept the liability risk of an un-authorized change of an account's registered email address
Request the user provide:
Verify the typed email addresses match each other -- and are valid -- and the current password is correct. DO NOT GO FURTHER unless all of these conditions are successfully met -- and instead show error messages explaining what is broken.
IF all of the above verification conditions are successfully met, THEN
Inform the user that they need to click on the link in BOTH email messages that were sent within .
When server receives web request with a nonce:
OTOH:
IF the nonce is NOT a known nonce for changing a registered email address
IF nonce IS known, BUT is outside the expiration time OR the change-registered-email-address process for that nonce IS complete:
(this could be written better, apologies, rushing as I'm under time pressure)
Q's that people might have:
WHY send any email address to the currently-registered email address?
WHY require the user to click a link in an email sent to the requested-new email address?
The text was updated successfully, but these errors were encountered: