bpo-1025395: Fix email.utils.parseaddr to handle multiple hops#142
bpo-1025395: Fix email.utils.parseaddr to handle multiple hops#142ioanatia wants to merge 1 commit intopython:masterfrom
Conversation
|
Hello, and thanks for your contribution! I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA). Unfortunately we couldn't find an account corresponding to your GitHub username on bugs.python.org (b.p.o) to verify you have signed the CLA. This is necessary for legal reasons before we can look at your contribution. Please follow these steps to help rectify the issue:
Thanks again to your contribution and we look forward to looking at it! |
350ccc1 to
9804620
Compare
Update the URLs for the documentation (http://stackless.readthedocs.io) and source code browsing (https://github.com/stackless-dev/stackless). (cherry picked from commit b4a5725)
|
Hey folks! I was wondering why my PR keeps failing, even though I did a rebase a couple of days ago! Can someone elucidate what might be going on here, please? From what I'm noticing, the Travis job has a different config than other jobs, so I'm kinda confused about why is that. Thanks a lot! |
|
It's totally fine to just do a merge if that's easier since we will do a squash commit in the end. I've restarted the Travis job just in case it was a weird fluke, though. |
9804620 to
6d17d16
Compare
|
Rebasing the branch on latest master does not fix it. |
|
Maybe try doing a merge instead of a rebase? We've had issues in the past of Travis only grabbing the last 50 commits so that can lead to issues with older PRs. |
Update the URLs for the documentation (http://stackless.readthedocs.io) and source code browsing (https://github.com/stackless-dev/stackless).
|
Thanks for the PR, @ioanatia.
The change seems unrelated to your change, but unless that got fixed we can't accept the PR. I realize this PR is quite old, so it's possible the fix for the failed travis build has already been done.
The instruction for blurb add is here |
Mariatta
left a comment
There was a problem hiding this comment.
Please see my review: #142 (comment)
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
8219850 to
4194c66
Compare
Update the URLs for the documentation (http://stackless.readthedocs.io) and source code browsing (https://github.com/stackless-dev/stackless). (cherry picked from commit b4a5725)


This fixes a very old issue with
email.utils.parseaddr.Right now
email.utils.parseaddrwill fail to parse the input correctly when the address contains a route with multiple hops. However it does not have an issue with single hop routes.RFC5322 states that including route hops is obsolete and that the route part should be ignored (Section 4.4). It also states that this syntax is valid and it must be accepted and parsed (Section 4).
https://bugs.python.org/issue1025395