Milter-greylist

Hi,

  Is it possible to "include" a config file with milter-greylist?  I'd
like to have a set of directive in another file than greylist.conf.

Regards,

Ugo

On Mon, Jul 17, 2006 at 03:36:16PM -0000, Ugo Bellavance wrote:
>   Is it possible to "include" a config file with milter-greylist?  I'd
> like to have a set of directive in another file than greylist.conf.

Implementing this leads to a small issue: we need to monitor all the
included files for changes (reminder: libmilter catches signals, so 
we cannot use SIGHUP to reload config, and we have to monitor config
file changes). That is possible for local files, but if we want to
include remote files, that would break.

-- 
Emmanuel Dreyfus
manu@...

--- In milter-greylist@yahoogroups.com, Emmanuel Dreyfus <manu@...> wrote:
>
> On Mon, Jul 17, 2006 at 03:36:16PM -0000, Ugo Bellavance wrote:
> >   Is it possible to "include" a config file with milter-greylist?  I'd
> > like to have a set of directive in another file than greylist.conf.
> 
> Implementing this leads to a small issue: we need to monitor all the
> included files for changes (reminder: libmilter catches signals, so 
> we cannot use SIGHUP to reload config, and we have to monitor config
> file changes). That is possible for local files, but if we want to
> include remote files, that would break.

I only use local config files, so that wouldn't be a problem.  Is it
hard to implement?

> 
> -- 
> Emmanuel Dreyfus
> manu@...
>

Ugo Bellavance <iolubik@...> wrote:

> I only use local config files, so that wouldn't be a problem.  Is it
> hard to implement?

The difficult point is that conf_parse() is not re-entrant. You cannot
just call it again from itself each time you see an include directive. 

That probably means you need to have two passes. First pass looks for
include directives, builds a flat file and second pass actually parse
the file.

So we would have to add a new parser, for instance in files called 
preconf_lex.l preconf_yacc.y. It would build a flat config in a
temporary file and a chained list of included files to monitor. 

Next step is to modify the config reload code:
1) monitor the list of included files
2) If a change was detected, create a temporary file with mkstemp
3) Call preconf_parse() 
4) Call conf_parse()
 
-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@...

>   Is it possible to "include" a config file with milter-greylist?  I'd
> like to have a set of directive in another file than greylist.conf.

Hi,

I generate the actual config file with a small script,
using a template config file and extracting the e-mail
addresses of the greylisting subscribed users from a
flat file database. I don't really need the include
directive.

Bests,
Attila

On Wed, Jul 19, 2006 at 10:13:40AM +0200, attila.bruncsak@... wrote:
> I generate the actual config file with a small script,
> using a template config file and extracting the e-mail
> addresses of the greylisting subscribed users from a
> flat file database. I don't really need the include
> directive.

Me too, FWIW...

-- 
Emmanuel Dreyfus
manu@...