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/
