[issue24066] send_message should take all the addresses in the To: header into account

[Kirill Elagin]

Kirill Elagin added the comment:

Oh, I see now.

It is a good idea to raise an error either in `send_message` or at the moment 
when a second `To`/`Cc`/`Bcc` header is added to the message.

--
resolution:  -> not a bug
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[R. David Murray]

R. David Murray added the comment:

Ah, but that is intentional.  Those fields can only appear once per message, 
per the RFC.  The new email API will raise an error if you attempt to add them 
more than once.  Perhaps we should raise an error instead of ignoring the 
duplicates, given that we are actively inspecting those headers.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[Kirill Elagin]

Kirill Elagin added the comment:

Ah, I’m so dumb. Of course the tests work as there are multiple addresses but 
still just one field.

Here is the test for multiple fields.

--
Added file: http://bugs.python.org/file39263/multiple_fields_test.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[R. David Murray]

R. David Murray added the comment:

Can you write a test that shows the failure?  There is an existing test that 
demonstrates sending to multiple addresses (testSendMessageWithAddresses), so 
you must have found an edge case that isn't tested.  (Either that, or the 
existing test is broken.)

--
stage: patch review -> test needed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[Ned Deily]

Changes by Ned Deily :


--
nosy: +barry, r.david.murray -ned.deily
stage:  -> patch review
versions: +Python 3.5

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[Kirill Elagin]

Kirill Elagin added the comment:

x_x

--
keywords: +patch
Added file: http://bugs.python.org/file39219/multiple_to.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[Ned Deily]

Ned Deily added the comment:

Kirill, the patch is missing.

--
nosy: +ned.deily

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue24066] send_message should take all the addresses in the To: header into account

[Kirill Elagin]

New submission from Kirill Elagin:

If I have a message with multiple `To` headers and I send it using 
`send_message` not specifying `to_addrs`, the message gets sent only to one of 
the recipients.

I’m attaching my patch that makes it send to _all_ the addresses listed in 
`To`, `Cc` and `Bcc`.

I didn’t add any new tests as the existing ones already cover those cases and I 
have no idea how on Earth do they pass.

--
components: Library (Lib)
messages: 242158
nosy: kirelagin
priority: normal
severity: normal
status: open
title: send_message should take all the addresses in the To: header into account

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com