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
