Message140527
| Author |
catalin.iacob |
| Recipients |
catalin.iacob, felipecruz, mpg, pablomouzo, r.david.murray |
| Date |
2011年07月17日.12:56:01 |
| SpamBayes Score |
2.616816e-05 |
| Marked as misclassified |
No |
| Message-id |
<1310907363.18.0.327801705853.issue7484@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I looked at the Felipe's patch and hopefully made some improvements.
Unlike Felipe's patch I didn't change the reply of the SMTP server in the tests but instead use what VRFY and EXPN actually send to index the users and lists dictionaries. If <> would be sent the lookup would fail. Similarly, when VRFY return 550 it echoed the address as received and now it's tested to be equal to something without <>.
By the way, but I was wondering:
* is the try/except really needed or just a historical artifact (why would email.utils.parseaddr raise AttributeError?)
* is the test to None correct? It was added by the fix to issue1430298 but does email.utils.parseaddr ever return None for the address? (I could only get it to return '')
I kept quoteaddr as is to make it easier to review the patch but if David confirms the above points are valid I can create new issues for them and simplify/fix quoteaddr. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2011年07月17日 12:56:03 | catalin.iacob | set | recipients:
+ catalin.iacob, r.david.murray, pablomouzo, mpg, felipecruz |
| 2011年07月17日 12:56:03 | catalin.iacob | set | messageid: <1310907363.18.0.327801705853.issue7484@psf.upfronthosting.co.za> |
| 2011年07月17日 12:56:02 | catalin.iacob | link | issue7484 messages |
| 2011年07月17日 12:56:01 | catalin.iacob | create |
|