Merge pull request #22 from kvdb/bccfixup

fix: don't let bcc recipients end up in 'to'.

flask_mailgun/api.py

@@ -37,7 +37,7 @@ def send(self, message, envelope_from=None):
         :param envelope_from: Email address to be used in MAIL FROM command.
         """
         mesage_data = {'from': envelope_from or message.sender,
-                       'to': message.send_to,
+                       'to': message.recipients,
                        'subject': message.subject,
                        "cc": message.cc,
                        "bcc": message.bcc,

flask_mailgun/message.py

@@ -70,10 +70,6 @@ def __init__(self, subject='',
         self.rcpt_options = rcpt_options or []
         self.attachments = attachments or []
 
-    @property
-    def send_to(self):
-        return set(self.recipients) | set(self.bcc or ()) | set(self.cc or ())
-
     @property
     def html(self):
         return self.alts.get('html')

tests/test_send.py

@@ -15,6 +15,7 @@ def test_send_simple_message(self):
         message = Message(subject="Hello",
                           sender="from@example.com",
                           recipients=["u1@example.com", "u2@example.com"],
+                          bcc=["foo@bar.com"],
                           body="Testing some Mailgun awesomness!")
         self.mailgun.send(message)
         self.assertTrue(self.mock_post.called)
@@ -26,7 +27,9 @@ def test_send_simple_message(self):
         self.assertEqual(auth, ('api', b'testtesttest'))
         # self.assertEqual(files, [])
         self.assertEqual(data['from'], message.sender)
-        self.assertEqual(data['to'], set(message.recipients))
+        self.assertEqual(data['to'], message.recipients)
+        self.assertEqual(data['bcc'], message.bcc)
+        self.assertEqual(data['cc'], [])
         self.assertEqual(data['subject'], message.subject)
         self.assertEqual(data['text'], message.body)