Closed Bug 886742 Opened 11 years ago Closed 11 years ago

Clicking Forgot pin does nothing if bango logout is an error page.

Categories

(Marketplace Graveyard :: Payments/Refunds, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
2013-08-13

People

(Reporter: scolville, Assigned: scolville)

References

Details

(Keywords: regression)

Attachments

(1 file)

Attached file Console.log
STR: 

 * Once you have set a pin (webpay not bango), try to buy an app
 * Instead of entering the pin click "Forgot your pin?"

What happens:

 * User gets stuck on loading screen
 * The only way out is to close the trusted-ui.

What should happen:

 * User can reset their pin.
Keywords: regression
This appears to be caused by http://mozilla.test.bango.org/mozpayments/logout/ being a 500.

We should however work out a better way to handle this because the current reponse causes logout to hang because the bango.logout() in the deferreds here: https://github.com/mozilla/webpay/blob/master/media/js/pay/pay.js#L126 doesn't resolve.

Seems also that with dataType = 'script' the response is eaten, whereas if the dataType is 'text' it does at least expose the error.
Depends on: 886760
Summary: Clicking Forgot pin does nothing → Clicking Forgot pin does nothing if bango logout is an error page.
The bango logout url is fixed now but we need to work out what we can do to show an error in the event of this happening again.

The biggest problem is when the 500 happened it was just an html page and as a result no errors arose as a result of the failed request.
Priority: -- → P2
Flags: affects-tricycle+
Flags: affects-seville+
Flags: affects-seahorse+
Flags: affects-moss+
Flags: affects-durango+
Assignee: nobody → scolville
Adding to this milestone because its a P2.
Target Milestone: --- → 2013-07-18
Target Milestone: 2013-07-18 → 2013-07-25
Target Milestone: 2013-07-25 → 2013-08-01
Target Milestone: 2013-08-01 → 2013-08-08
Status: NEW → ASSIGNED
Target Milestone: 2013-08-06 → 2013-08-13
https://github.com/mozilla/webpay/commit/082de6665060dbbb17f781d6790f8665de994660
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: