Milter-greylist

On Sat, Dec 11, 2004 at 10:19:14AM +0100, manu@... wrote:
> Ivan F. Martinez <ml@...> wrote:
> 
> > The patch is available :              
> > http://www.saisp.br/ifm/patches/milter-greylist.c.patch
> > 
> > Also a m4 file to configure milter-greylist, and a sample Rule to get
> > data from access.db
> > http://www.saisp.br/ifm/patches/milter-greylist.m4
> > 
> > I'm accepting suggestions to make it better.
> 
> FWIW, Ivan sent me the patch behind the scene, and I'd like some
> feedback from other contributors. Especially from Remy Card for ACL
> interraction.
> 
> I wonder if there woulnd't be some benefit to expcitely pull various
> whitelist/greylist methods from the ACL. Something like this:
> 
> acl whitelist sendmaildb 
> acl greylist default

	This can easily be done but this requires some changes in the ACL 
API (ctx has to be added as a parameter to acl_filter() to enable the use
of sendmail data in this function).

> That would enable mixing greylist.conf ACL with sendmail DB. The same
> trick could be used for SPF. I wonder if it would make sense to do the
> same for SMTP auth or authowhitelist, as those would always tend to be
> thefirst items in the ACL.
> 
> Opinions? Don't hesitate to tell me that it's useless and too
> complicated :-)

	Well, this is certainly not too complicated.  Sendmail DB check, SPF
and SMTP auth can also be moved in the ACL code.  I do not know about
autowhitelist since this feature should not be optional, IMHO.

	BTW, I think that the new ACL scheme should be tested a bit more
before we move existing tests in it.  Any comments from testers?  Is anyone
using the new ACL feature?  Is it working in environments different from
mine?

		R\ufffdmy