httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: load order dependency between mod_ldap and mod_authnz_ldap
Date Mon, 04 Jul 2011 16:48:49 GMT
On Mon, Jul 04, 2011 at 11:43:33AM +0200, Graham Leggett wrote:
> I have already stated the basis for the veto: every single apparent
> flaw in the apr_ldap code that caused wrowe to remove it from APR is
> still present in the code that wrowe dumped into httpd. 

It's incumbent on you to provide specific technical objections if 
vetoing code, not this hand-waving "objections must exist because of X".

The removal from APR was because the API does not match design of the 
other APIs in APR, because it is an "incomplete abstraction" - mod_ldap 
must use the underlying LDAP API directly as well as the wrapper 
functions.  This is not something which translates directly across to 
httpd; the exported API from mod_ldap was already an incomplete 
abstraction, adding more exports does not change that.  Indeed, it 
explains the motivation for consolidating this stuff in one place.

If you have specific concerns with exposing all the ap[r]_ldap_* API 
from httpd, then that is something we can maybe resolve; it looks like 
mod_authnz_ldap and mod_vhost_ldap only require the url_parse hook, so 
maybe we could cut the additional exported API back down to that?

You allude to build issues without giving details, what's still broken? 
(Thanks Stefan for fixing my own screwup)

Regards, Joe


Mime
View raw message