ACK.
Please make sure to complete the remaining work as indicated.
Ade
On Thu, 2013-10-31 at 19:21 -0500, Endi Sukma Dewata wrote:
On 10/28/2013 11:41 AM, Endi Sukma Dewata wrote:
> On 10/10/2013 10:00 AM, Endi Sukma Dewata wrote:
>> Previously the GroupMemberProcessor could only run on CA because the
>> base Processor class depends on CA. To fix the problem all CA-specific
>> code in Processor has been moved into a new CAProcessor. All existing
>> subclasses of Processor will inherit from CAProcessor, but the
>> GroupMemberProcessor will continue to inherit from Processor.
>
> Reposted as separated patches. The first one renames Processor to
> CAProcessor to reflect the current CA dependency issue. The second one
> creates a new generic Processor super class to fix GroupMemberProcessor
> issue. Please apply in this order: 331, 326-1.
>
> As Ade noted, there are additional works to be done:
> 1. Some other non-CA-specific fields/methods should be moved to the
> super class as well including stats, getSSLClientCertificat(), maybe the
> authenticate().
> 2. SecurityDomainProcessor will need to run on a standalone KRA, so it
> should not inherit from CAProcessor either.
New patch #326-2 attached. Fixed GroupMemberProcessor to extend from
Processor.
_______________________________________________
Pki-devel mailing list
Pki-devel(a)redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel