[Pki-devel] [pki-devel][PATCH] 0069-Show-KeyOwner-info-when-viewing-recovery-requests.patch

[Christina Fu]

while the patch works, I think the original code logic is somehow flawed 
in a way that it uses the "profile" attribute to determine whether the 
request was non-TMS archival requests, and if null it treats it as TMS.  
It would make better sense if we add a separate case instead of lumping 
the handling of recovery requests inside where the TMS handling is at.

thanks,
Christina

On 06/01/2016 03:13 PM, John Magne wrote:
> Show KeyOwner info when viewing recovery requests.
>      
>      This simple fix will grab the subject info out of the cert
>      associated with either pending or complete recovery requests being
>      viewed in the KRA UI.
>      
>      For example:
>      
>      KeyOwner:  UID=jmagne, O=Token Key User
>      
>      Will be displayed.
>      Have seen this display for both pending and completed recovery requests.
>
>      This simple fix should be good enough for this round, despite the bug
>      asking about agent info and such. Those enhancements for later.
>      
>      Ticket : Ticket #1512 : Key owner info missing from the Search results of Recovery request
>
>
> _______________________________________________
> 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/20160603/67167643/attachment.htm>