[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