intermediate certificate preloading healer
Categories
(Core :: Security: PSM, enhancement, P1)
Tracking
()
People
(Reporter: keeler, Assigned: keeler)
References
Details
(Whiteboard: [psm-assigned])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Users may have cached intermediate certificates in their cert9.db that have also been downloaded via intermediate preloading. Some operations in NSS are linear (or worse) with the number of certificates in cert9.db, so we should consider removing those superfluous entries (as long as the user hasn't modified their trust bits). This could be done as a background task that periodically scans cert9.db and removes certificates as appropriate.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
In general, PSM caches intermediates from verified certificate chains in the
NSS certdb. Before bug 1619021, this would include preloaded intermediates,
which is unnecessary because cert_storage has a copy of those certificates, and
so they don't need to take up time and space in the NSS certdb. This patch
introduces the intermediate preloading healer, which periodically runs on a
background thread, looks for these duplicate intermediates, and removes them
from the NSS certdb.
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 3•4 years ago
|
||
Backed out for xpcshell failures on test_intermediate_preloads.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/ce51bacc88393c714df6b27f4831a37a65db930d
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=305151853&repo=autoland&lineNumber=2733
Assignee | ||
Comment 4•4 years ago
|
||
Thanks. Working on the underlying issue in bug 1644224.
Comment 6•4 years ago
|
||
bugherder |
Comment 7•4 years ago
|
||
Is this something we should consider uplifting to Beta so it's in the next ESR or can this fix ride 79 to release?
Assignee | ||
Comment 8•4 years ago
|
||
Since the next ESR won't have had intermediate preloading without also having bug 1619021, it's not as important. I think it would be best to be cautious and let this ride the trains.
Updated•4 years ago
|
Description
•