Closed
Bug 843397
Opened 11 years ago
Closed 11 years ago
Don't make users who are logged into mktplace log in to webpay
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
2013-03-14
People
(Reporter: krupa.mozbugs, Assigned: kumar)
References
Details
(Keywords: regression, Whiteboard: p=2)
steps to reproduce: 1. Go to the details page for a paid app 2. Click on the purchase button 3. Sign in using Identity expected behavior: The purchase flow starts and user goes to the PIN screen observed behavior: The purchase flow starts and the user ends up in the "Sign in" screen for Webpay.
Reporter | ||
Updated•11 years ago
|
Severity: normal → major
Whiteboard: p=
Comment 1•11 years ago
|
||
I remember Kumar saying this had to do with: https://github.com/mozilla/webpay/blob/master/webpay/settings/base.py#L232 Perhaps he knows more about it.
Flags: needinfo?(kumar.mcmillan)
Assignee | ||
Comment 2•11 years ago
|
||
We need to fix this. I'm not positive it's to do with that new setting. It may have to do with us not specifying loggedInUser in webpay.
Flags: needinfo?(kumar.mcmillan)
Priority: -- → P2
Comment 3•11 years ago
|
||
An undefined loggedInUser with an existing persona session should result in onlogin() being called which should just send the assertion to us and everything is good. But it appears it is calling onlogout() which, if loggedInUser is undefined should only happen if person thinks there is no session.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kumar.mcmillan
Target Milestone: 2013-02-28 → 2013-03-14
Updated•11 years ago
|
Version: 1.1 → 1.2
Updated•11 years ago
|
Whiteboard: p= → p=2
Assignee | ||
Comment 4•11 years ago
|
||
It looks like there is a Persona bug around forced issuer which is causing this. As a workaround, I disabled issuer forcing. That fixes this specific bug but it should be re-enabled at some point. See bug 850084
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•