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)

Avenir
x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: andy+bugzilla, Assigned: ashort)

References

Details

Attachments

(2 files, 1 obsolete file)

Attached video reset-pin.mov (deleted) —
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.
Attached file logout-log.txt (obsolete) (deleted) —
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.
Attached file logout-log.txt (deleted) —
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.
Assignee: nobody → ashort
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
Priority: -- → P2
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.

Attachment

General

Creator:
Created:
Updated:
Size: