Milter-greylist

Alvaro Marín alvaro@... [milter-greylist]
<milter-greylist@yahoogroups.com> wrote:

> Is there any limitation to add multiple headers with racl rules?

Yes, there is a limitation. Currently we store the headers to add in a
single-valued field. That happens in acl_actions() from acl.c here:

        if (ap->ap_addheader) {
                priv->priv_sr.sr_addheader = strdup(ap->ap_addheader);
                if (priv->priv_sr.sr_addheader == NULL) {
                        mg_log(LOG_ERR, "strdup failed");
                        exit(EX_OSERR);
                }
         (...)

Supporting multiple addheader clauses, either on a single ACL or on
multiple ones, requires changing the priv_sr.sr_addheader field to be a
chained list (see LIST_INIT/LIST_INSERT_HEAD/LIST_FOREACH/LIST_REMOVE
macros for examples). That seems an average-level contribution. Feel
free to work on it, your patch will be welcome.


-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@...