This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author r.david.murray
Recipients barry, r.david.murray, terry.reedy, torsten.becker
Date 2011-03-27.18:51:49
SpamBayes Score 4.9373984e-08
Marked as misclassified No
Message-id <1301251910.09.0.418304375752.issue1690608@psf.upfronthosting.co.za>
In-reply-to
Content
You should check if 'charset' is a string, and call Charset on it only if it is (a Charset may be passed directly in other email package interfaces, and so should be supported here as well.

The test doesn't need to cater for the fact that either b or B (or q or Q) are legitimate: we know which one the package is generating, so just test for that.

For the Message['To'], I wasn't clear.  What I would like is a test that includes non-ascii characters in the address part, *without* passing it through formataddr, to see what the package currently does with it.  This may in fact reveal an additional bug.  But, it is really out of scope for this issue, so you can just remove that test (sorry).

There should also be an update to the docs (Doc/library/email.utils.rst) documenting the API change.
History
Date User Action Args
2011-03-27 18:51:50r.david.murraysetrecipients: + r.david.murray, barry, terry.reedy, torsten.becker
2011-03-27 18:51:50r.david.murraysetmessageid: <1301251910.09.0.418304375752.issue1690608@psf.upfronthosting.co.za>
2011-03-27 18:51:49r.david.murraylinkissue1690608 messages
2011-03-27 18:51:49r.david.murraycreate