cyrus-imapd 2.1.15, sieve, lmtpd, and return-path header

[Rob Siemborski]

On Tue, 23 Sep 2003, Chris Stromsoe wrote:

> On Tue, 23 Sep 2003, Rob Siemborski wrote:
>
> > Ah, I missed that (or forgot).  X-Sieve is added as part of
> > func->addheaders, so it might be a bit more interesting...
>
> Is addheaders used for anything other than the X-Sieve header?  Was it
> ever intended to be used for more?

No.

> If it was only ever intended to be used to add the X-Sieve header, is
> there any objection to removing func->addheaders and redoing the X-Sieve
> header generation with the other headers in lmtpengine.c?

Yes.  You need to remember that lmtpengine.c is used for both lmtpd and
lmtpproxyd.  The former needs to add the X-Sieve header, the later does
not.  This can't be done inside of lmtpengine.c.

> If it was intended to be a generic "headers to add to every message"
> interface, do you have any objection to broadening it into an array of
> struct Header {}, with code to add arbitrary headers from imapd.conf?

This wouldn't be a problem.

-Rob

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Rob Siemborski * Andrew Systems Group * Cyert Hall 207 * 412-268-7456
Research Systems Programmer * /usr/contributed Gatekeeper