Re: [Pki-devel] Fwd: [pli-devel][PATCH] 0081-Fix-for-Add-ability-to-disallow-TPS-to-enroll-a-sing.patch

2016-10-10 Thread John Magne
ACKED by cfu and she verbally acked a quick additon to eh fix for #1664 Pushed to master. - Original Message - From: "Christina Fu" To: pki-devel@redhat.com Sent: Friday, October 7, 2016 5:06:51 PM Subject: Re: [Pki-devel] Fwd: [pli-devel][PATCH]

Re: [Pki-devel] Fwd: [pli-devel][PATCH] 0081-Fix-for-Add-ability-to-disallow-TPS-to-enroll-a-sing.patch

2016-10-07 Thread Christina Fu
Code looks good. One suggestion. Since we have to appease to the current NSS way of looking up certs, how about making the default true so that it will keep the old encryption certs by default? Of course we are taking up more space now on the token when it's true, so we should plan to

[Pki-devel] Fwd: [pli-devel][PATCH] 0081-Fix-for-Add-ability-to-disallow-TPS-to-enroll-a-sing.patch

2016-10-07 Thread John Magne
Actually attach the patch. - Forwarded Message - From: "John Magne" To: "pki-devel" Sent: Friday, October 7, 2016 11:45:17 AM Subject: [pli-devel][PATCH] 0081-Fix-for-Add-ability-to-disallow-TPS-to-enroll-a-sing.patch Fix for: Add ability to