email reporter: Allow multiple recipients for sendmail method (#797, …

…by monperrus)

Co-authored-by: Thomas Perl <[email protected]>

CHANGELOG.md

@@ -18,6 +18,7 @@ The format mostly follows [Keep a Changelog](http://keepachangelog.com/en/1.0.0/
 
 ### Fixed
 
+- `email` reporter: Allow multiple recipients for `sendmail` method (#797, by monperrus)
 - Fix documentation for watching Github tags and releases, again (#723)
 - Fix `--test-reporter` command-line option so `separate` configuration option is no longer ignored when sending test notifications (#772, by marunjar)
 - Fix line height and dark mode regression (#774 reported by kongomongo, PRs #777 and #778 by trevorshannon)

docs/source/reporters.rst

@@ -61,7 +61,7 @@ The list of built-in reporters can be retrieved using::
 At the moment, the following reporters are built-in:
 
 - **discord**: Send a message to a Discord channel
-- **email**: Send summary via e-mail / SMTP
+- **email**: Send summary via e-mail / SMTP / sendmail
 - **ifttt**: Send summary via IFTTT
 - **mailgun**: Send e-mail via the Mailgun service
 - **matrix**: Send a message to a room using the Matrix protocol
@@ -290,6 +290,21 @@ public Matrix room, as the messages quickly become noisy:
      minimal: true
      enabled: true
 
+E-Mail via sendmail
+---------------------
+
+You can send email via the system's ``sendmail`` command provided by the MTA. You need to set ``method: sendmail`` in the config file:
+
+.. code:: yaml
+
+    report:
+      email:
+        enabled: true
+        from: '[email protected]'
+        to: '[email protected]'
+        method: sendmail
+
+
 E-Mail via GMail SMTP
 ---------------------
 

lib/urlwatch/mailer.py

@@ -112,7 +112,7 @@ def __init__(self, sendmail_path):
         self.sendmail_path = sendmail_path
 
     def send(self, msg):
-        p = subprocess.Popen([self.sendmail_path, '-oi', '-f', msg['From'], msg['To']],
+        p = subprocess.Popen([self.sendmail_path, '-oi', '-f', msg['From']] + msg['To'].split(','),
                              stdin=subprocess.PIPE,
                              stderr=subprocess.PIPE,
                              universal_newlines=True)