1241650 - remove nsIX509CertDB.findCertNicknames

Status: RESOLVED FIXED

(Core :: Security: PSM, defect)

Description

[Dana Keeler (she/her) [:keeler]]

nsIX509CertDB.findCertNicknames is not used in mozilla-central or comm-central. There are a small handful of addons that use it, but as far as I can tell they only do so as a way of listing known certificates. nsIX509CertDB.getCerts is a much better API for that, so it should be used instead.

Comment 1

[Dana Keeler (she/her) [:keeler]]

Attachment: MozReview Request: bug 1241650 - remove nsIX509CertDB.findCertNicknames r?mgoodwin

Review commit: https://reviewboard.mozilla.org/r/35529/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/35529/

Comment 2

[Mark Goodwin [:mgoodwin]]

Comment on attachment 8720974 [details]
MozReview Request: bug 1241650 - remove nsIX509CertDB.findCertNicknames r?mgoodwin

https://reviewboard.mozilla.org/r/35529/#review32271

Looks good to me.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5a61b51e47b0

Comment 4

[Dana Keeler (she/her) [:keeler]]

Thanks!

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/5a61b51e47b0