[Tickets #12938] Wrong Sieve implementation of "Do not send responses to bulk or list messages"

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12938
------------------------------------------------------------------------------
  Ticket             | 12938
  Created By         | skhorde at smail.inf.fh-bonn-rhein-sieg.de
  Summary            | Wrong Sieve implementation of "Do not send responses to
                     | bulk or list messages"
  Queue              | Ingo
  Version            | 3.1.3
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


skhorde at smail.inf.fh-bonn-rhein-sieg.de (2014-01-27 08:29) wrote:

This script is generated, if you check "Do not send responses to bulk  
or list messages".

  1: # Sieve Filter
     2: # Generated by Ingo (http://www.horde.org/apps/ingo/)  
(01/27/2014, 09:23)
     3: require ["vacation", "regex"];
     4: # Vacation
     5: if allof ( not exists ["list-help", "list-unsubscribe",  
"list-subscribe", "list-owner", "list-post", "list-archive",  
"list-id", "Mailing-List"], not header :comparator "i;ascii-casemap"  
:is "Precedence" ["list", "bulk", "junk"], not header :comparator  
"i;ascii-casemap" :matches "To" "Multiple recipients of*" ) {
     6:     vacation :days 7 :addresses "/dev/null at example.com"  
:subject "Test" "Test 1";
     7: }

RFC 5228 sec. 5.5 explains, that "All of the headers must exist or the  
test is false". Hence, if one of the enumerated headers is missing in  
the exists clause, e.g. "Mailing-List", the "not exists" is false.  
E.g. the messages of the Horde list to not contain a "Mailing-List"  
header.

I would suggest to change the condition to:

anyof ( not exists "list-help", not exists "list-unsubscribe", not  
exists "list-subscribe", ...