Closed
Bug 867282
Opened 11 years ago
Closed 11 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•11 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•11 years ago
|
Assignee: nobody → amckay
Assignee | ||
Updated•11 years ago
|
Assignee: amckay → nobody
Flags: needinfo?(ndesaulniers)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → amckay
Assignee | ||
Updated•11 years ago
|
Target Milestone: --- → 2013-07-25
Assignee | ||
Comment 2•11 years ago
|
||
https://github.com/mozilla/receiptverifier/pull/18
Target Milestone: 2013-07-25 → 2013-08-08
Comment 3•11 years ago
|
||
Merged /me still has to fix in simulator https://github.com/mozilla/r2d2b2g/issues/748
Flags: needinfo?(ndesaulniers)
Comment 4•11 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•11 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•11 years ago
|
||
Nope, sorry I thought it was required.
Assignee | ||
Updated•11 years ago
|
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
•