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.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Python-LDAP-dev mailing list
Python-LDAP-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/python-ldap-dev

Reply via email to