> https://bugzilla.mozilla.org/show_bug.cgi?id=315871 > >> (https://bugzilla.mozilla.org/attachment.cgi?id=230307)...? This would >> add support for exporting a cert in PKCS#7 format from Cert Viewer. > > Do you have previews for the UI changes that patch makes?
This particular patch ("getPKCS7(), third version") doesn't include any UI changes - it just adds a GetPKCS7() method to nsNSSCertificate and exposes it through nsIX509Cert3. I'd especially welcome comments from NSS developers if I'm using libsmime properly. > Does that patch put the features of your extension info FireFox? Not this one, but the first two attached to this bug ("patch v1", "proposal for patch v2"). The bug has a somewhat convoluted history: - In November '05, it was filed as an RFE by Kai. - In May '06, Kai asked whether it was ok to use code from my Cert Viewer Plus extension (I wasn't aware of this bug entry until then, I had released the first version in February). Of course I welcomed Kai's initiative. - In June, Kai attached the first version of the patch (https://bugzilla.mozilla.org/attachment.cgi?id=225016), which is based on v0.7 of my code, but only adds the "Save To File" button to the UI of Firefox/Thunderbird/Seamonkey (the reason for omitting the "Display In PEM Format" button being that it didn't work with Seamonkey) - Still in June, I submitted an enhanced version of Kai's patch (https://bugzilla.mozilla.org/attachment.cgi?id=225283), which adds the "Display In PEM Format" button supporting Seamonkey, too. At that time, I also asked whether it would be feasible to add a scriptable getPKCS7() method to PSM to allow saving in PKCS#7 format. - Finally, in July, I submitted a separate patch for adding just this method. I soon realized that the first and the second version didn't really do the right thing, so that's why "getPKCS7(), third version" is now the version I consider the proper one. > You might attach a preview to that bug, or add a comment with a URL > for the preview to it. Would it make sense to create a separate bug entry for the getPKCS7() patch, since this is actually a backend-only thing? Kaspar _______________________________________________ dev-tech-crypto mailing list dev-tech-crypto@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-tech-crypto