[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#7118) [PATCH] Improve SASL plugin handling

On Tuesday 10 January 2012 09:56:02 Pierangelo Masarati wrote:
> > Now, if ldapdb is incompatible with slapd at a fundamental level, and
> > causes problems when loaded, I think not to load it is the right thing
> > to do, and this is what my patch does.
> An alternative solution consists in creating a specific folder 
> containing plugins one wants to load, and start slapd with SASL_PATH set 
> to point to that folder.  This allows fine tuning of what to load 
> without modifying a single line of code.

That is certainly true, however, it misses the point I am trying to make.
I want to provide a fix for the problem I encountered.

>From a user's perspective, it is certainly preferable for the whole thing
to 'just work', instead of being provided with instructions about how to
configure one's system to avoid problems that arise when using the standard
configuration. This places the burden on the user, for no other gain than
being able to load a module that cannot be used anyway. With the change I
proposed, slapd can work with every standard cyrus-sasl installation right
away, no special configuration required, and the problem is solved once
and forever.