fix requests not being sent to the cc recipients

From a SMTP pov, the To and Cc header are not read. The real recipient is stated at the begining of a send mail transaction. It's up to the client to collect this recipient list. Sendmail does this by reading the To and Cc header. smtplib doesn't do this when being invoked with an explicit To array. This commit adds the Cc people to the recipient array. The issue has been introduced by the switch from sendmail to smtplib.

aurweb/scripts/notify.py

@@ -78,7 +78,7 @@ class Notification:
             msg['Subject'] = self.get_subject(lang)
             msg['From'] = sender
             msg['Reply-to'] = reply_to
-            msg['To'] = to
+            msg['To'] = self.get_to()
             if self.get_cc():
                 msg['Cc'] = str.join(', ', self.get_cc())
             msg['X-AUR-Reason'] = reason
@@ -135,6 +135,9 @@ class ResetKeyNotification(Notification):
         else:
             return [(self._to, self._lang)]
 
+    def get_to(self):
+        return self._to
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Password Reset', lang)
 
@@ -183,6 +186,9 @@ class CommentNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._recipients
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Comment for {pkgbase}',
                                     lang).format(pkgbase=self._pkgbase)
@@ -227,6 +233,9 @@ class UpdateNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._recipients
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Package Update: {pkgbase}',
                                     lang).format(pkgbase=self._pkgbase)
@@ -275,6 +284,9 @@ class FlagNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._recipients
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Out-of-date Notification for '
                                     '{pkgbase}',
@@ -314,6 +326,9 @@ class OwnershipEventNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._recipients
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Ownership Notification for {pkgbase}',
                                     lang).format(pkgbase=self._pkgbase)
@@ -349,6 +364,9 @@ class ComaintainershipEventNotification(Notification):
     def get_recipients(self):
         return [(self._to, self._lang)]
 
+    def get_to(self):
+        return self._to
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Co-Maintainer Notification for '
                                     '{pkgbase}',
@@ -393,6 +411,9 @@ class DeleteNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._to
+
     def get_subject(self, lang):
         return self._l10n.translate('AUR Package deleted: {pkgbase}',
                                     lang).format(pkgbase=self._old_pkgbase)
@@ -445,6 +466,9 @@ class RequestOpenNotification(Notification):
     def get_recipients(self):
         return [(self._to, 'en'), self._cc]
 
+    def get_to(self):
+        return self._to
+
     def get_cc(self):
         return [row[0] for row in self._cc]
 
@@ -505,6 +529,9 @@ class RequestCloseNotification(Notification):
     def get_recipients(self):
         return [(self._to, 'en'), self._cc]
 
+    def get_to(self):
+        return self._to
+
     def get_cc(self):
         return [row[0] for row in self._cc]
 
@@ -553,6 +580,9 @@ class TUVoteReminderNotification(Notification):
     def get_recipients(self):
         return self._recipients
 
+    def get_to(self):
+        return self._recipients
+
     def get_subject(self, lang):
         return self._l10n.translate('TU Vote Reminder: Proposal {id}',
                                     lang).format(id=self._vote_id)