Handle auto-submitted header on inbound email like bulk.

Part of issue2550564.

Author: rouilj

CHANGES.txt

@@ -20,6 +20,9 @@ Fixed:
   [A-z][A-z0-9_]+[A-z_]. This was never documented or enforced, but
   we get obscure errors if the rules are not followed. (Tom
   Ekberg tests by John Rouilard)
+- issue2550564 - Roundup sets "Precedence: bulk" on all outgoing mail,
+  which seems wrong. Handle Auto-Submitted header on *inbound* email
+  like we do precedence bulk. This is part of this issue.
 
 2020-07-13 2.0.0
 

roundup/mailgw.py

@@ -535,10 +535,18 @@ def handle_ignore(self):
         ''' Check to see if message can be safely ignored:
             detect loops and
             Precedence: Bulk, or Microsoft Outlook autoreplies
+            Auto-Submitted not equal to no, so auto-generated,
+            auto-replied and auto-notified are dropped. rstrip
+            auto-submitted value because trailing comment/whitespace
+            is allowed per RFC3834. Note that we only handle whitespace.
+            Lowercase the value as references say No and no as possible
+            values.
         '''
         if self.message.get_header('x-roundup-loop', ''):
             raise IgnoreLoop
         if (self.message.get_header('precedence', '') == 'bulk'
+                or self.message.get_header('auto-submitted', 'no').rstrip().lower() \
+                         != 'no'
                 or self.subject.lower().find("autoreply") > 0):
             raise IgnoreBulk
 

test/test_mailgw.py

@@ -3590,6 +3590,37 @@ def testItsBulkStupid(self):
 Hi, I'm on holidays, and this is a dumb auto-responder.
 ''')
 
+    def testItsAutoSubmittedStupid(self):
+        self.assertRaises(IgnoreBulk, self._handle_mail,
+            '''Content-Type: text/plain;
+  charset="iso-8859-1"
+From: Chef <[email protected]>
+Auto-Submitted: Auto-Generated
+To: [email protected]
+Cc: [email protected]
+Message-Id: <dummy_test_message_id>
+Subject: Re: [issue] Testing...
+
+Hi, I'm on holidays, and this is a dumb auto-responder.
+''')
+
+    def testItsHumanSubmitted(self):
+        ''' keep trailing spaces on Auto-submitted header
+        '''
+        self.db.keyword.create(name='Foo')
+        self._handle_mail('''Content-Type: text/plain;
+  charset="iso-8859-1"
+From: richard <[email protected]>
+To: [email protected]
+Message-Id: <followup_dummy_id>
+In-Reply-To: <dummy_test_message_id>
+Auto-submitted:   No  
+Subject: [keyword1] Testing... [name=Bar]
+
+''')
+        self.assertEqual(self.db.keyword.get('1', 'name'), 'Bar')
+
+
     def testAutoReplyEmailsAreIgnored(self):
         self.assertRaises(IgnoreBulk, self._handle_mail,
             '''Content-Type: text/plain;