Closed
Bug 1487258
Opened 6 years ago
Closed 6 years ago
load enterprise roots on a background thread
Categories
(Core :: Security: PSM, enhancement, P1)
Core
Security: PSM
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: keeler, Assigned: keeler)
References
Details
(Whiteboard: [psm-assigned])
Attachments
(1 file)
At startup, if the enterprise roots pref is enabled, we do the finding/loading on the main thread. Since this can involve certificate verification, it should probably be done on a background thread. Luckily we already have a handy background thread that we use to load the loadable roots module, so this should be a straightforward change.
Assignee | ||
Comment 1•6 years ago
|
||
Loading enterprise roots could potentially take a while, so we certainly shouldn't do it on the main thread at startup. Note that this doesn't address the case where a user enables the feature while Firefox is running. This isn't great but since it's an about:config preference rather than a first-class preference exposed in about:preferences, we can probably get away with it for now.
Comment 2•6 years ago
|
||
Comment on attachment 9005284 [details] bug 1487258 - load enterprise roots on a background at startup r?jcj J.C. Jones [:jcj] (he/him) has approved the revision.
Attachment #9005284 -
Flags: review+
Assignee | ||
Comment 3•6 years ago
|
||
Thanks! https://treeherder.mozilla.org/#/jobs?repo=try&revision=cfc6701931c5abd98348fffcd54f733c774a8789
Pushed by dkeeler@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3a800dfc6ad8 load enterprise roots on a background at startup r=jcj
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3a800dfc6ad8
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•