Re: [PATCH] default values

[<mi...@st...>]

Geert Jansen wrote:
> 
> the attached patch allows you to use default values for the
> functions/arguments below. This is helpful in an environment where
> /etc/openldap/ldap.conf is kept up to date with correct information.

Sorry, I won't accept this patch for general use. Note: This behaviour
is not guaranteed to work in future versions of python-ldap. In the
future there MAY be a change to another/additional LDAP client lib which
does not have a ldap.conf at all.

I consider python-ldap to rather provide a low-level API. Applications
implemented on top of python-ldap should deal with whatever config file
they like passing all required parameters around. Hiding config file
access within such a low-level module is IMO bad design.

Ciao, Michael.