Update user accounts to be keyed by FxA uid upon FxA login

RESOLVED FIXED in 2014-10-07

Status

P2
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: ashort, Assigned: robhudson)

Tracking

2014-10-07
x86
Mac OS X
Points:
---

Details

(Whiteboard: [qa-])

(Reporter)

Description

4 years ago
Firefox Accounts provides a unique ID for users; we should store it as a key for user accounts, and use it for account lookup instead of email. This will let us automatically handle the case where a Firefox Account user changes his email address.
(Reporter)

Comment 1

4 years ago
This has to wait until there is a production deployment of fxa-oauth-server with stable UIDs.
Priority: -- → P2
(Reporter)

Comment 2

4 years ago
We should look up users by UID. If the UID from FxA isn't found, look up the email address. If an account is found, update its username field to the FxA UID.

(If the UID is found and the email address doesn't match, the user changed his email address and the email address should be updated.)
Assignee: ashort → robhudson.mozbugs
(Assignee)

Comment 3

4 years ago
https://github.com/mozilla/zamboni/commit/a883b69 

I think it'd be great to verify that an existing FxA login works, then change your email in FxA, then log in again on Marketplace and ensure that you are still the user you thought you were by making sure you own the same apps, etc.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Target Milestone: --- → 2014-10-07

Comment 4

4 years ago
I am not sure how we can change the email address for an FxA account. In Account Settings page (https://marketplace.allizom.org/settings) the Email field can't be edited.
Flags: needinfo?(robhudson.mozbugs)

Comment 5

4 years ago
Changing email would be done to your Firefox Account, so at: https://accounts.firefox.com/

That feature hasn't been added to Firefox Accounts yet, though, so I'm not sure how much you can QA at this point.
Flags: needinfo?(robhudson.mozbugs)

Comment 6

4 years ago
Marking this [qa-] based on comment 5
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.