diff options
author | Jefferson Stachelski <jstachel@thoughtworks.com> | 2015-11-25 18:07:20 -0200 |
---|---|---|
committer | Jefferson Stachelski <jstachel@thoughtworks.com> | 2015-11-25 18:07:20 -0200 |
commit | 1c6614f80a3107003d88d9a66c8af19e69b52f6d (patch) | |
tree | 7dba05786d474e84b9c0c48753dc97a752f23bcb | |
parent | 811aa0b5850344f65e0b16831f537143a5c461ad (diff) |
Isse #515 - Refactored bcc login on send mail
-rw-r--r-- | service/pixelated/adapter/services/mail_sender.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/service/pixelated/adapter/services/mail_sender.py b/service/pixelated/adapter/services/mail_sender.py index acd9f034..8ccfc5a2 100644 --- a/service/pixelated/adapter/services/mail_sender.py +++ b/service/pixelated/adapter/services/mail_sender.py @@ -64,16 +64,18 @@ class MailSender(object): deferreds = [] for recipient in recipients: - if recipient in bccs: - mail.headers['Bcc'] = [recipient] - else: - mail.headers['Bcc'] = [] - + self._define_bcc_field(mail, recipient, bccs) smtp_recipient = self._create_twisted_smtp_recipient(recipient) deferreds.append(outgoing_mail.send_message(mail.to_smtp_format(), smtp_recipient)) return defer.DeferredList(deferreds, fireOnOneErrback=False, consumeErrors=True) + def _define_bcc_field(self, mail, recipient, bccs): + if recipient in bccs: + mail.headers['Bcc'] = [recipient] + else: + mail.headers['Bcc'] = [] + def _build_error_map(self, recipients, results): error_map = {} for email, error in [(recipients[idx], r[1]) for idx, r in enumerate(results)]: |