apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: [PATCH] LDAP extension to apr-util (take 2)
Date Mon, 13 Aug 2001 20:03:07 GMT
On Fri, Aug 10, 2001 at 12:06:37AM +0200, Graham Leggett wrote:
> Hi all,
> 
> Here is another snapshot of the library that incorporates LDAP into
> apr-util.
> 
> This patch is big, but it's an incorporation of a body of new code
> which works as a unit.
> 
> The build should work correctly both with the default configuration
> (LDAP not included) to the configuration --with-ldap. Please test this
> for me to be sure.
> 
> The header file apr_ldap.h incorporates some high level access to a
> number of LDAP functions, like opening connections, checking userids,
> and comparing users. Access is also possible to any of the ldap-*()
> routines within the LDAP SDK.
> 
> The source code is half a vastly modified version of Dave Carrigan's
> auth_ldap module, published under the ASF licence. The other half takes
> the form of mod_auth_ldap, which is being posted separately to the
> new-httpd list.
> 
> Currently the code works for me (Linux/PPC) - would be keen to see what
> other people's experiences are. It still needs a lot of work - but at
> this point feedback will be valuable.
> 
> Comments...?


I probably missed some conversation while I was away in July, but this is an
awfully large chunk of code and functionality to put into APRUTIL. I don't
see how this is sufficiently generic, or a set of utilities. LDAP is a
really large item with lots of scariness to get things right. I'm worried
that this limited functionality is going to grow without bounds over time.

I'm all for a new apr-ldap CVS module / library. But its presence in APRUTIL
feels very questionable to me.

Cheers,
-g

p.s. if there was a long conversation about this in July, then a reference
would be great, and I'll go read the thread(s)

-- 
Greg Stein, http://www.lyra.org/

Mime
View raw message