[sork] Re: [PATCH] new ldap driver for passwd

[Roel Gloudemans]

> Le Lundi, 14. Février 2005 19:48, Roel Gloudemans a écrit :
>> Hi Philipp,
>>
>> To do it right I think the choice how to log on should be configurable.
>
> Of cause the driver is configurable, you can choose which driver to use in
> "backend.php". I just coded an additional driver which gives you more choice.

Ah..., but the code is very similar. It maybe better to merge it with 
the existing code.

>> The shadow parameters are for password policies. Why should they be
>> removed? (I for one use them to make passwords expire and to warn the
>> user when this is going to happen)
>
> I know they are useful, but I don't use them and they give me error messages
> because the fields the function looks for don't exist in my directory. And
> there is no option to switch shadow off. I thought about adding the option,
> but couldn't be bothered as I don't use it. If you need it feel free to
> include it.

If we do it right errors can be prevented. Also see the first remark

>> On a broader perspective: Maybe it is better to e.g. merge passwd and
>> accounts. There are a few functions you would like to have on both
>> sides (it test I have even have an accounts module that can talk to an
>> AD; no passwords yet though, but you would need to have admin access to
>> the directory). Cheers,
>> Roel.
>
> I don't want to manage accounts through horde, so for me just passwd is quite
> all right. There could be uses of a merged project, I don't know too much
> about it, as long as I can switch off the account part.
>
I don't either. The accounts module just shows your account details, 
like your full name, your homedirectory and when your password must 
change. I should say merging it with passwd will make one better module.

I'm very busy at the moment, but when I have time I will do some coding 
myself and merge your LDAP sriver with the existing one (unless someone 
beats me to it) Anyone got any words about merging passwd with accounts?

Cheers,
Roel.