Skip to content
This repository has been archived by the owner. It is now read-only.

Allow users to change their account email address #2527

Closed
zaach opened this issue Jun 4, 2015 · 12 comments
Closed

Allow users to change their account email address #2527

zaach opened this issue Jun 4, 2015 · 12 comments

Comments

@zaach
Copy link
Contributor

@zaach zaach commented Jun 4, 2015

Users should be able to change their account email address to prevent them from getting locked out if they forget their password and lose access to their old email account.

This depends on: mozilla/fxa-auth-server#489

@ryanfeeley has the UX for this.

/cc @eoger

@zaach
Copy link
Contributor Author

@zaach zaach commented Jun 4, 2015

@ryanfeeley post the mockups here when you get a chance.

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Jun 7, 2015

@zaach - Can you explain more about the need for this feature? Which password is being forgot in this scenario, the email provider password?

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Jun 7, 2015

Also, we have to ensure reliers do not depend on the user's email address as the unique identifier, or else the user's relier account will be inaccessible.

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Jun 7, 2015

Finally, how will this interact with a user who has a Sync session? Any session for that matter, will changing an email address invalidate all outstanding session tokens?

@ryanfeeley
Copy link
Contributor

@ryanfeeley ryanfeeley commented Jun 7, 2015

Hi Shane: email addresses are not permanent for all users, therefore we should allow users to change their email. If we don't there will be a small portion of forgetful users who will be unable to log in.

@eoger eoger self-assigned this Jun 8, 2015
@eoger eoger added the WIP label Jun 8, 2015
@ckarlof ckarlof added waffle:ready and removed WIP labels Jun 8, 2015
@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Jun 8, 2015

Next steps are to break down the work for this with a github checklist.

@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Jun 8, 2015

@eoger, I believe we're using the WIP label for work-in-progress PRs. When you're ready to start this self-assign, and move into waffle:in progress.

@ryanfeeley
Copy link
Contributor

@ryanfeeley ryanfeeley commented Jul 22, 2015

@ckarlof @eoger @shane-tomlinson @zaach I am starting to think that we should first require that users verify their original email address as part of the process. Would be more secure for people who still control over their original email, but the downside is that for users who lost control over their original email address, they would not be able to regain control.

@billmaggs
Copy link

@billmaggs billmaggs commented Aug 13, 2015

I think that's an easy one; most people will still control their email account, and for those people who have lost control of what is probably a principal email account, they've probably got bigger problems than losing access to Sync data.

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Feb 16, 2016

This is FxA-28 on Aha, closing here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants