Message133696
| 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? |
|