Re: Sieve: vacation from: parameter with display-name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 08.03.16 um 14:19 schrieb Edda via Info-cyrus:
[...]

From: <My Name <myaddr_at_example.com>>

To my understanding of RFC 5322 this is not a valid „mailbox“, because of the outer angle brackets around. sendmail rewrites it internally to the resulting valid but ugly form:

From: "<My Name" <myaddr at example.com>

I looked at RFC 5322 a bit further. It states that address header fields are (finally) of the form "mailbox":

   mailbox         =   name-addr / addr-spec

where

   name-addr       =   [display-name] angle-addr

(addr-spec has no angle brackets around).

MTAs mostly tolerate the forms with angle brackets, but they are not compliant to RFC... as far as I see...

Sieve vacation already uses name-addr or addr-spec, so I would suggest this simple patch:

diff -Naur cyrus-imapd-2.4.18.orig/imap/lmtp_sieve.c cyrus-imapd-2.4.18/imap/lmtp_sieve.c --- cyrus-imapd-2.4.18.orig/imap/lmtp_sieve.c 2015-07-06 05:38:29.000000000 +0200 +++ cyrus-imapd-2.4.18/imap/lmtp_sieve.c 2016-03-10 13:44:04.173224594 +0100
@@ -650,8 +650,8 @@
     fprintf(sm, "Date: %s\r\n", datestr);

     fprintf(sm, "X-Sieve: %s\r\n", SIEVE_VERSION);
-    fprintf(sm, "From: <%s>\r\n", src->fromaddr);
-    fprintf(sm, "To: <%s>\r\n", src->addr);
+    fprintf(sm, "From: %s\r\n", src->fromaddr);
+    fprintf(sm, "To: %s\r\n", src->addr);
     /* check that subject is sane */
     sl = strlen(src->subj);
     for (i = 0; i < sl; i++)



Best regards,
Edda





----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
To Unsubscribe:
https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus




[Index of Archives]     [Cyrus SASL]     [Squirrel Mail]     [Asterisk PBX]     [Video For Linux]     [Photo]     [Yosemite News]     [gtk]     [KDE]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux