Re: [PATCH] smime_keys: Handle certificate chains in add_cert. (closes #3339)

2015-05-23 Thread Kevin J. McCarthy
Kevin J. McCarthy wrote: > Ah. I see my mistake. I didn't realize mutt was prompting for the > label first. The current patch added a prompt for *each* certificate > leaf+chain, but that won't work. I will change it to prompt only once > for the whole thing. Here's a revised patch loosening up

Re: [PATCH] smime_keys: Handle certificate chains in add_cert. (closes #3339)

2015-05-23 Thread Kevin J. McCarthy
Hi David, Thanks for testing my patch so quickly. David J. Weller-Fahy wrote: > It does handle all the certificates, but does not end up using the label > entered in mutt for all the certificates, nor does it allow the user to > enter labels for the certificates individually when used from within

Re: [PATCH] smime_keys: Handle certificate chains in add_cert. (closes #3339)

2015-05-23 Thread David J. Weller-Fahy
* David J. Weller-Fahy [2015-05-23 13:52 -0400]: Second, the request: For some reason the patch results in the intermediate *and* root certificates ending up in the file in which the intermediate certificate should be placed. In this context "the patch" refers to my patch on top of your patch

Re: [PATCH] smime_keys: Handle certificate chains in add_cert. (closes #3339)

2015-05-23 Thread David J. Weller-Fahy
* Kevin J. McCarthy [2015-05-22 17:21 -0400]: Attached is a patch which hopefully implements better certificate chain handling for add_cert. Although I've tried to test it, I am not a S/MIME user and don't have access to a large number of certificates to play with. I would greatly appreciate i