Yahoo Groups archive

Milter-greylist

Index last updated: 2026-04-28 23:32 UTC

Message

Re: [milter-greylist] Re: Multiple MX questions

2005-12-12 by Emmanuel Dreyfus

On Mon, Dec 12, 2005 at 03:48:02AM -0000, Ugo Bellavance wrote:
> > You can patch the milter to replace tempfail by success...
> 
> Yes... thanks.  But I'm not much of a programmer and I don't really
> want to mess anything up.  Would it be possible to implement this as a
> parameter in a future release?

Sure, if someone does it. Since you desire the feature, you seem to be
the right person for implementing it. And that's a good opportunity to 
learn about programming.

Quickly hacking it is not difficult, edit miller-greylist.c, and replace
SMFIS_TEMPFAIL by SMFIS_SUCCESS. You can even do a nicer hack by not changing
the occurence after the malloc failure (when you are out of memory, you 
temporarily reject until things get better).

The next step is to add a configuration option, which is a bit more complex,
but after all you just have to replicate what is already done for other 
options, such as noaccessdb. You have to edit the following files, duplicate
the lines where you have noaccessdb and replace noaccessdb by learnmode:
conf_lex.l conf_yacc.y conf.c conf.h

The remaining part is straightforward: replace your SMFIS_SUCCESS change by 
if (conf.c_learnmode) 
	return SMFIS_SUCCESS;
else
	return SMFIS_TEMPFAIL;

After this, be nice and add this new option to the man page (greylist.conf.5),
and contribute the result. 

-- 
Emmanuel Dreyfus
manu@...

Attachments

Move to quarantaine

This moves the raw source file on disk only. The archive index is not changed automatically, so you still need to run a manual refresh afterward.