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@redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel