<div dir="ltr"><div>Thanks for the feedback, I simply expected that addresses and <br></div><div>simple strings were treated differently.</div><div><br></div><div>Thorsten<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 28, 2018 at 1:55 PM, Steffen Kaiser <span dir="ltr"><<a href="mailto:skdovecot@inf.h-brs.de" target="_blank">skdovecot@inf.h-brs.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<span class=""><br>
<br>
On Mon, 28 May 2018, Thorsten Hater wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I stumbled upon the following behaviour of Pigeonhole, which I consider<br>
to be problematic. A user deployed a Sieve script similar to the following<br>
snippet<br>
<br>
if not anyof (address :is ["from","cc"] ["...", ..., "...@...<br>
GARBAGE", ...] {<br>
 fileinto "inbox.Trash";<br>
 stop;<br>
}<br>
<br>
Note the extra line break before GARBAGE. This script is obviously broken,<br>
but gets accepted by sievec and only fails later, at runtime with<br>
<br>
line X: error: found stray carriage-return (CR) character in quoted<br>
   string started at line X.<br>
<br>
So, the question is whether line breaks in strings are allowed in general<br>
and the runtime error is unavoidable, or should sievec return an error?<br>
</blockquote>
<br>
</span><a href="https://www.ietf.org/rfc/rfc3028.txt" rel="noreferrer" target="_blank">https://www.ietf.org/rfc/rfc30<wbr>28.txt</a> first hit of quoted-string<br>
<br>
 quoted-string = DQUOTE *CHAR DQUOTE<br>
           ;; in general, \ CHAR inside a string maps to CHAR<br>
           ;; so \" maps to " and \\ maps to \<br>
           ;; note that newlines and other characters are all allowed<br>
           ;; strings<br>
<br>
So, it's correct. But the address should reject the CR. I guess, Pigeonhole triggers the error for sanity purpose?<br>
<br>
- -- Steffen Kaiser<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1<br>
<br>
iQEVAwUBWwvuQsQnQQNheMxiAQILoA<wbr>gAyRjSObVJkrAmxzyLau9gIvvMOM2R<wbr>++HP<br>
pwsptIQ72xoYJOO/Lnd1TmfKTE9QYw<wbr>tOGkSKr8tiJVD8JOpL5fUbB6mZNOTX<wbr>kAv0<br>
TOW2gA7v06nXq6K0ETum8anoKTIF0o<wbr>4j5aQJ5yQ5CrzlVQqUwTsf4mVVNqK0<wbr>hn/L<br>
X5RAuCVQyx6sdvCB+lSOGmLv/fT8+x<wbr>HS03U6jzCp/Yov5OKsT29oOOF6dXWR<wbr>49Iw<br>
BL+DOd9T37hHF6ENp4A5wxX6iCMKLs<wbr>WL0f5xTcxwRK5GOiCDoUH6ZpiywD0P<wbr>tCuT<br>
VlusmbIByGON7foNlCPusTVcfq8Gen<wbr>MhOrgFcbp1PfRrShIQgsjWSg==<br>
=vgR1<br>
-----END PGP SIGNATURE-----<br>
</blockquote></div><br></div>