[Freeipa-devel] Re: [PATCH] Add HBAC plugin and introduce GeneralizedTime parameter type.
Pavel Zuna
pzuna at redhat.com
Fri Oct 2 14:29:06 UTC 2009
Rob Crittenden wrote:
> Pavel Zuna wrote:
>> Sumit noticed, that the HBAC plugin didn't make it into the tree and
>> the patch got lost in the depths of freeipa-devel.
>>
>> I reworked the plugin to meet the current standards and changed
>> GeneraliSedTime to GeneraliZedTime. For now, I put the new parameter
>> type into a separate file in ipalib/plugins/generalizedtime.py. When
>> it's tested enough, we're going to move it to ipalib/parameters.py.
>>
>> The patch also modifies bootstrap-template.ldif to create a cn=hbac
>> container after installing IPA.
>>
>> Pavel
>
> Shouldn't GeneralizedTime be in ipalib?
>
> I think GeneralizedTime should have some unit tests, same for hbac.
>
> rob
Update:
- Now includes extensive unit test for HBAC plugin.
- GeneralizedTime was moved to ipalib/parameters.
- Minor updated to HBAC plugin, mostly esthetics.
Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Add-HBAC-plugin-and-introduce-GeneralizedTime-parame.patch
Type: application/mbox
Size: 27425 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20091002/9b3c9d46/attachment.mbox>
More information about the Freeipa-devel
mailing list