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 torsten.becker
Recipients r.david.murray, sdaoden, torsten.becker
Date 2011-04-13.21:42:21
SpamBayes Score 1.0076189e-07
Marked as misclassified No
Message-id <BANLkTiktd95wHehWnrYX-XTA7Gnkwv4sbw@mail.gmail.com>
In-reply-to <1302722093.81.0.651536462549.issue11783@psf.upfronthosting.co.za>
Content
> OK, so when I went to apply this, I figured out that the patch isn't quite right.  I've redone the doc updates, and am attaching a version of the patch containing them.
>
> The issue is that the place that the IDNA decode support needs to be added isn't in parseaddr, it's in _parseaddr.py's AddresslistClass.  Tests are then needed to make sure that the IDNA decoding gets done both when parseaddr and getaddresslist are used.
>
> Do you want to tackle this, Torsten?

I would like to, but I probably will not get to it before Monday.  So
if anybody wants to work on this before that time, please feel free to
fix it properly. :)

Just two questions for the implementation:
  1. Would it be fine to move the helper _encode_decode_addr() into
_parseaddr.py and then import it in util.py, so it can be shared
between the two?
  2. Would line 232 in _parseaddr.py (AddrlistClass.getaddrlist) be a
good place to integrate it?
History
Date User Action Args
2011-04-13 21:42:22torsten.beckersetrecipients: + torsten.becker, r.david.murray, sdaoden
2011-04-13 21:42:21torsten.beckerlinkissue11783 messages
2011-04-13 21:42:21torsten.beckercreate