Closed
Bug 867282
Opened 13 years ago
Closed 12 years ago
Update receiptverifier to check product
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
2013-08-06
People
(Reporter: andy+bugzilla, Assigned: andy+bugzilla)
References
Details
The receipt verifier library currently doesn't check the product URL. As per bug 865498, I think we should do that. So this is about adding these checks to the library:
https://github.com/mozilla/receiptverifier
Comment 1•13 years ago
|
||
This will break how the simulator installs packaged apps, since the current design is to use garbage for the manifest_url API parameter. So when this changes, lets make sure someone working on the simulator is aware.
Updated•13 years ago
|
Assignee: nobody → amckay
| Assignee | ||
Updated•12 years ago
|
Assignee: amckay → nobody
Flags: needinfo?(ndesaulniers)
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → amckay
| Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → 2013-07-25
| Assignee | ||
Comment 2•12 years ago
|
||
Target Milestone: 2013-07-25 → 2013-08-08
Comment 3•12 years ago
|
||
Merged
/me still has to fix in simulator
https://github.com/mozilla/r2d2b2g/issues/748
Flags: needinfo?(ndesaulniers)
Comment 4•12 years ago
|
||
Hey Andy, is this live yet? I just double checked with packaged apps and nothing appears broken.
Flags: needinfo?(amckay)
| Assignee | ||
Comment 5•12 years ago
|
||
Merged: https://github.com/mozilla/receiptverifier/commit/f11274
The check is optional, so nothing should break, is there something you are expecting to break?
Flags: needinfo?(amckay)
Comment 6•12 years ago
|
||
Nope, sorry I thought it was required.
| Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•