Closed
Bug 1119428
Opened 10 years ago
Closed 10 years ago
Reset PIN logged me out and hung
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: andy+bugzilla, Assigned: ashort)
References
Details
Attachments
(2 files, 1 obsolete file)
On 2.1 device.
1. Login
2. Try to purchase an app
3. Click reset PIN
Expected:
Enter my password into Firefox Accounts.
Actual:
Flashes a persona message.
Completely logs me out of Firefox Accounts.
Hangs on a spinner.
Taking videos on the phone is hard, all but the last step occurred on simulator:
Seems like bug 1103103 wasn't actually resolved.
I'll add dealing with removing the reset click page in the click page removal branch.
Or actually it's probably going to be better to handle it independently for easier review.
So this bug is because NativeFxA logout isn't working correctly (Or rather it's not working like the old persona code expects - the old persona code was re-jigged for native FxA to use here: https://github.com/mozilla/spartacus/commit/d743620ee4213798611e96b89a39bbeb67cbabf3). So we're hanging at the logout which never resolves.
Not sure what the native API's expectations are here. Clearly this is a point where the native FxA API differs from Persona.
Adding a log which appears to show logout getting fired twice by indentity on the native API.
Not sure if it's expected we should call navigator.mozId.logout.
Attachment #8546701 -
Attachment is obsolete: true
I think this is why things go strange as logout handlers are called twice at our end as we are responding to 2 onlogout events.
Comment 7•10 years ago
|
||
We also need to make sure that logging out of Marketplace (during Forgot PIN flow) doesn't log the user out of the phone. More details in https://bugzilla.mozilla.org/show_bug.cgi?id=1073177
Reporter | ||
Updated•10 years ago
|
Priority: -- → P2
Assignee | ||
Comment 8•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•