Message344389
| Author | jpic |
|---|---|
| Recipients | Dain Dwarf, Windson Yang, barry, bortzmeyer, cnicodeme, jpic, kal.sze, maxking, msapiro, ned.deily, nicoe, r.david.murray, vstinner, xtreak |
| Date | 2019-06-03.07:51:50 |
| SpamBayes Score | -1.0 |
| Marked as misclassified | Yes |
| Message-id | <CAC6Op1_QpNxpuY6x=HsEnLmq1_y82VTg5khfi0vt_rWQ-xdsag@mail.gmail.com> |
| In-reply-to | <1559416833.73.0.0656305198822.issue34155@roundup.psfhosted.org> |
| Content | |
|---|---|
Thanks for your explanation, but in perspective with other invalid
domains, such as "foo." currently resulting in an empty string too:
>>> email.message_from_string('From: a@foo.',policy=email.policy.default)['from'].addresses[0].domain
''
Do you think this should also change the value of domain to "foo." ?
Also yes with parseaddr it seems that domain is an empty string if it
didn't find a valid domain at all, which is pretty safe in case of
malicious injection attempt - if that's what we're trying to save
python programs from, a clarification of the objective of the patch
would be welcome. |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2019-06-03 07:51:50 | jpic | set | recipients: + jpic, barry, vstinner, msapiro, ned.deily, r.david.murray, nicoe, maxking, kal.sze, Windson Yang, xtreak, cnicodeme, bortzmeyer, Dain Dwarf |
| 2019-06-03 07:51:50 | jpic | link | issue34155 messages |
| 2019-06-03 07:51:50 | jpic | create | |