Closed Bug 960837 Opened 6 years ago Closed 6 years ago

Use the basic receipt check introduced in bug 757226 for installations

Categories

(Core Graveyard :: DOM: Apps, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla29

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file, 1 obsolete file)

As we've introduced these checks, we may as well use them for installations.

I've only a doubt, should the installation fail when one of the receipts is invalid or when all of them are invalid?
Attached patch Tentative patch (obsolete) — Splinter Review
Attaching a tentative patch that takes the first approach (the installation fails if one of the receipts is invalid).
When any one of them is invalid, in my opinion.
Attached patch PatchSplinter Review
Attachment #8361414 - Attachment is obsolete: true
Attachment #8361972 - Flags: review?(fabrice)
Attachment #8361972 - Flags: review?(fabrice) → review+
https://hg.mozilla.org/mozilla-central/rev/25caedc66f90
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Depends on: 964877
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.