Closed
Bug 976192
Opened 11 years ago
Closed 11 years ago
Block Android payments till we are ready
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2014-04-08
People
(Reporter: andy+bugzilla, Assigned: kumar)
References
Details
(Whiteboard: [A4A])
Attachments
(1 file)
468.87 KB,
image/jpeg
|
Details |
Its possible, although highly unlikely that an end user hits webpay from an in-app payment on an app. That's because the mozPay API has landed, but we aren't ready to get on with doing payments yet because there are bugs to be fixed.
This code would check the user agent to see if its Android and block the payment with a nice error message. We'll turn this on for prod and off for stage so we can test.
When Android payments are done, we'll delete all this code.
Reporter | ||
Updated•11 years ago
|
Priority: -- → P2
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kumar.mcmillan
Target Milestone: --- → 2014-03-25
Assignee | ||
Comment 1•11 years ago
|
||
disabled but only in prod https://github.com/mozilla/webpay/commit/971a3892fb496d305e245103ede9e1a9022e9de0
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 2•11 years ago
|
||
Please add STR here or mark it with [qa-] if no QA is needed.
Flags: needinfo?
Comment 3•11 years ago
|
||
Using http://inapp-pay-test.paas.allizom.org/, I was able to get to the credit card screen.
I will check again after tomorrow's push.
Flags: needinfo?
Assignee | ||
Comment 4•11 years ago
|
||
STR: using the in-app tester, try to make a purchase on Android. You should see an error that says the payments are temporarily disabled.
Comment 5•11 years ago
|
||
I have tried to make a payment using the in-app tester http://inapp-pay-test.paas.allizom.org/ and I was able to make a purchase on my Android tablet using the Movistar SIM that I have in the Inari. Is this expected?
Flags: needinfo?
Comment 6•11 years ago
|
||
(In reply to Iulian Timis from comment #5)
> I have tried to make a payment using the in-app tester
> http://inapp-pay-test.paas.allizom.org/ and I was able to make a purchase on
> my Android tablet using the Movistar SIM that I have in the Inari. Is this
> expected?
On Android, payments should be blocked.
Status: RESOLVED → REOPENED
Flags: needinfo?
Resolution: FIXED → ---
Assignee | ||
Comment 7•11 years ago
|
||
I see the PAY_DISABLED message on my Android phone. I don't have my Android tablet set up right now. Can you tell me what the user agent is from Firefox Nightly? This page will show it: http://people.mozilla.org/~kmcmillan/ua.html Short URL: http://mzl.la/1d0eM3q
Flags: needinfo?(iulian.timis)
Comment 8•11 years ago
|
||
Mozilla/5.0 (Android; Tablet; rv:31.0) Gecko/31.o Firefox/31.0
I am able to make a payment on Prod, and on Stage I get "error: PAY_REQUEST_ERROR_NO_VALID_REQUEST_FOUND"
Flags: needinfo?(iulian.timis)
Assignee | ||
Comment 9•11 years ago
|
||
Oh, good catch. Thanks.
This should fix it https://github.com/mozilla/webpay/commit/4fc1e48eab56f75a747d5da1e12ec7df2000969b
However, this change won't be on prod until April 8th
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: 2014-03-25 → 2014-04-08
Comment 10•11 years ago
|
||
I have tried making a payment again using http://inapp-pay-test.paas.allizom.org/ on Stage, but I get the same error message "error: PAY_REQUEST_ERROR_NO_VALID_REQUEST_FOUND" on Nightly 31.0a1 (Android 4.2.1) Galaxy Nexus.
Flags: needinfo?(kumar.mcmillan)
Assignee | ||
Comment 11•11 years ago
|
||
Have you installed this testing add-on? https://addons.mozilla.org/en-US/firefox/addon/dev-marketplace/ You need this so that the stage server is whitelisted as a place that can accept payments. However, as of a couple hours ago, I think this change was deployed to production.
Flags: needinfo?(kumar.mcmillan)
Comment 12•11 years ago
|
||
Verified as fixed for production. I get "Payments are temporarily disabled". On stage, payments for Android are working.
Attaching postfix screenshot.
Closing bug.
Status: RESOLVED → VERIFIED
Comment 13•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•