Stephen Gallagher wrote:
On 08/31/2009 02:56 PM, Dmitri Pal wrote:
> Simo Sorce wrote:
>> Since we enumerate in background and use the member/memberof scheme you
>> can't do group enumerations without also doing a user enumeration
first.
>>
>> So turn the enumeration option into a boolean with warnings if the old
>> syntax is used.
>>
>> Simo.
>>
>>
> Simo, I think this patch is missing the part where it reads the INI file
> and sticks initial data into the database.
> It should have same logic:
> * try to read as int, if success then probably stick as int in the sysdb
> * otherwise try to read as bool and stick bool in.
We discussed this, but currently we read all entries as strings into the
confdb (since that's how they're stored internally) and then we do the
validation when we query the confdb instead.
Once the INI parser supports schema validation, we'll move these checks
there.
Ah, Ok, makes sense then.
I do not know the code in details so I do not feel myself qualified to
ack it but looks Ok at the first glance.
>>
------------------------------------------------------------------------
>>
>> _______________________________________________
>> sssd-devel mailing list
>> sssd-devel(a)lists.fedorahosted.org
>>
https://fedorahosted.org/mailman/listinfo/sssd-devel
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel
--
Thank you,
Dmitri Pal
Engineering Manager IPA project,
Red Hat Inc.
-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/