add support for sending to multiple recipients via sendmail

CHANGELOG.md

@@ -18,6 +18,7 @@ The format mostly follows [Keep a Changelog](http://keepachangelog.com/en/1.0.0/
 
 ### Fixed
 
+- Fix multiple recipients for sendmail reporter (#797)
 - 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

@@ -69,6 +69,7 @@ At the moment, the following reporters are built-in:
 - **prowl**: Send a detailed notification via prowlapp.com
 - **pushbullet**: Send summary via pushbullet.com
 - **pushover**: Send summary via pushover.net
+- **sendmail**: Pipe a message to the system sendmail command
 - **shell**: Pipe a message to a shell command
 - **slack**: Send a message to a Slack channel
 - **stdout**: Print summary on stdout (the console)

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)