Closed Bug 1244357 Opened 6 years ago Closed 6 years ago

Shim the certificate database in xpcshell tests

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox46 --- fixed
firefox47 --- fixed

People

(Reporter: mossop, Assigned: mossop)

Details

Attachments

(1 file)

The xpcshell tests generate and modify a lot of add-ons on the fly. When signing becomes mandatory they will suddenly all fail to work unless we also sign them. But switching from generated extensions to in-tree ones would be a lot of work  even ignoring getting them signed.

In xpcshell tests though we have the ability to shim certain xpcom components and the certificate database we use to verify XPI signatures is one of them. This can't work for the other test suites because it has already been loaded by the time the test harness runs.

With this approach we can effectively force the add-ons manager to believe that add-ons are signed even though they have no signature.
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/33101/diff/1-2/
Attachment #8714474 - Flags: review?(rhelmer)
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/33101/diff/2-3/
Attachment #8714474 - Flags: review?(rhelmer)
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

https://reviewboard.mozilla.org/r/33101/#review30473
Attachment #8714474 - Flags: review?(rhelmer) → review+
Attachment #8714474 - Attachment description: MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r?rhelmer → MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/33101/diff/3-4/
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

Can't figure out how to re-request review in review board. Can you skim over the last interdiff, it just makes the cache flush test ignore flushes from the fake certificate db.
Attachment #8714474 - Flags: review+ → review?(rhelmer)
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

https://reviewboard.mozilla.org/r/33101/#review30873
Attachment #8714474 - Flags: review?(rhelmer) → review+
https://hg.mozilla.org/mozilla-central/rev/1d5b210de3c9
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

Approval Request Comment
[Feature/regressing bug #]: Signed add-ons
[User impact if declined]: This is needed to force signing on in 46
[Describe test coverage new/current, TreeHerder]: On m-c for a while
[Risks and why]: Very low, mostly test only changes
[String/UUID change made/needed]: None
Attachment #8714474 - Flags: approval-mozilla-aurora?
Comment on attachment 8714474 [details]
MozReview Request: Bug 1244357: Use a shim around the certificate DB to allow the add-ons manager to think that add-ons are signed when they aren't. r=rhelmer

We want our tests to work for sure. Approved for uplift to aurora.
Attachment #8714474 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.