> On Tue, Jan 24, 2006 at 08:16:55AM -0600, Corbey, Clayton wrote:
> > -----Original Message-----
> > From: Manfred [mailto:[EMAIL PROTECTED]
> > Sent: Tuesday, January 24, 2006 07:10
> > To: [email protected]
> > Subject: Re: rcpthosts
> >
> > >> Why doesn qmail-ldap not do lookups for all hosts in rcpthosts?
> > >>
> > >> None of my users are on the qmail system, or a qmail system for that
> > >> matter. I want the lookups to be done and then the email forwaded (if
> > >> the lookup succedeed) to another server(Exchange, boo) internally
> >
> > after
> >
> > >> that. How can I configure qmail-ldap to do this?
> > >>
> > >> -Clayton
> > >
> > >I'm using the nice little patch from Eric Hoeve (on list in Jun 2005)
> >
> > which
> >
> > >checks RCPTCHECK also for non local users over LDAP.
> > >One installation has 2 qmail-ldap and one scanner smtp-gateway on one
> > >server
> > >in a row. One qmail-ldap is used for in front check with rbl,
> >
> > rcptcheck, >etc.
> >
> > >than virus, spam and finally local delivery.
> > >
> > >BTW: I think it would be a nice option to stock qmail-ldap
> > >
> > >Manfred
> > >
> > >--- qmail-1.03/qmail-smtpd.old     2005-06-21 14:26:59.000000000 -0500
> > >+++ qmail-1.03/qmail-smtpd.c       2005-06-21 14:27:24.000000000 -0500
> > >@@ -1114,9 +1114,9 @@
> > >   if (rcptcheck) {
> > >     if (!goodmailaddr()) {
> > >       logline(4,"recipient verify, recipient not in goodmailaddr");
> > >-      if (addrlocals()) {
> > >+      if (addrallowed()) {
> > >   char *s;
> > >-  logline(4,"recipient verify, recipient is local");
> > >+  logline(4,"recipient verify, recipient is allowed");
> > >         switch (ldaplookup(addr.s, &s)) {
> > >           case 1: /* valid */
> > >       logline(4,"recipient verify OK");
> >
> > Right on Manfred!
> >
> > I am going to apply this patch and here's to hoping for the best!
>
> This patch will not be added to the qmail-ldap tree because it breaks
> current behaviour. With the patch it is no longer possible to use rcpthost
> for domains that are not part of you ldap tree.
> If something like this should go in it needs to be correctly designed and
> not break current setups.

Hi Claudio,

didn't mean that this should be the standard configuration for qmail-ldap, 
clearly this would cause trouble to most installations.
I just thought that having this as a documented compile _option_ it would save 
a lot of people time and trouble to get this type of functionality which is 
really neat with just this little patch. Reading the list this topic is 
probably often needed in "front line qmail-ldap setup" which is supposed to 
drop unknown ldap-users at smtp level and then forward mail to further mail 
processing stations rather than feeding all the crap through the rest of the 
mail setup.
Most people (like me) are probably comfortable compiling the system with 
different options set but still have the trust to use "stock qmail-ldap" 
rather than patching third person patches without enough C 
knowledge/expertise/experience to be sure not to break something severe.
Wouldn't it be possible to add this type of feature as compile or 
configuration option? It looks like a really tiny change in the code (that's 
why I went with the patch).
Just my $.02

Keep up the great work with qmail-ldap, love it!

Manfred

Reply via email to