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

[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 revert it when/if NSS changes.

conditional ACK if you do that.

Christina


On 10/07/2016 02:01 PM, John Magne wrote:
> Actually attach the patch.
>
> ----- Forwarded Message -----
> From: "John Magne" <jmagne at redhat.com>
> To: "pki-devel" <pki-devel at redhat.com>
> 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 disallow TPS to enroll a single user on multiple tokens. #1664
>      
>      This bug was previously not completely fixed where we left a loophole to allow a user to
>      end up with 2 active tokens. This fix closes that loophole.
>      
>      Also:
>      
>      Fix for: Unable to read an encrypted email using renewed tokens. #2483
>      
>      This fix provides for a new optional renewal based token policy, that
>      allows the user to retain or recover old encryption certs for that profile,
>      that get overwritten by the renewal process.
>      
>      An example is:
>      
>      RENEW=YES;RENEW_KEEP_OLD_ENC_CERTS=YES
>      
>      The second part of the policy is new.
>      
>      When this is set to "YES", the system will make sure the old enc cert
>      will remain on the token. If it's missing or "NO", no such attempt will be made.
>
>
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pki-devel/attachments/20161007/9bd724de/attachment.htm>