python-ldap-dev

[PATCH] default values

[Geert J. <ge...@bo...>]

Hi,

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.

   ldap.initialize(uri)
   LDAPObject.search_*(base, scope)

The semantics are identical to the command-line "ldapsearch" tool. This 
means:

- uri <- "HOST" from /etc/openldap.conf
- base <- "BASE" from /etc/openldap.conf
- scope <- SCOPE_SUBTREE

The patch is against the latest CVS and has been tested.

Regards, Geert

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.