[Pki-devel] [PATCH] 91 Refactored CertRevokeRequest and CertUnrevokeRequest classes in Dogtag 10

[Christina Fu]

Hi Abhishek,
The code appears to be correct, provided that the previously refactored 
code (which I did not review) works correctly, and it does not break the 
non-REST code.  Could you please

1. provide a signed audit log event for one revocation request from the 
cli, where it shows an actual request id and verify that it is indeed 
the correct request id?
2.perform one single revocation from the non-REST agent interface and 
then verify the same revocation log event type for log request id?

thanks,
Christina

On 05/05/2014 03:48 AM, Abhishek Koneru wrote:
> Sorry for the spam!
> Please ignore the previous email.
>
> --Abhishek
> On Mon, 2014-05-05 at 06:44 -0400, Abhishek Koneru wrote:
>> Please review the patch which refactors the CertRevokeRequest class and
>> removes the CertUnrevokeRequest class in Dogtag 10. Description of the
>> patch:
>>
>> There seems to be no use of the requestID parameter in both revoke
>> and unrevoke request. Removed requestID attribute in CertRevokeRequest
>> remove the class CertUnrevokeRequest.
>>
>> Also made changes in RevocationProcesor to use the requestID of the
>> request created in it.
>>
>> The setRequestID() is being called in the DoRevoke and DoUnRevoke
>> servlets.
>> Removed the call and a function auditRequesterId in both the classes.
>>
>> The auditRequestorId method tries to get a "requestID" stored as a INPUT
>> field
>> in the reasonToRequest page. The ReasonToRevoke class which generates
>> this page does not set the value.
>>
>> * This patch is required for patch 92. The unrevoke_request method in
>> CertClient on the python side will not work without this patch.
>>
>> --Abhishek
>> _______________________________________________
>> Pki-devel mailing list
>> Pki-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/pki-devel
>
>
> _______________________________________________
> 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/20140515/e5c9a8bc/attachment.htm>