Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islingtonmiss-islington commented Jul 17, 2019

Before:

 >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='a', domain='malicious.org'),) >>> parseaddr('[email protected]@important.com') ('', '[email protected]') After: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='', domain=''),) >>> parseaddr('[email protected]@important.com') ('', 'a@') 

https://bugs.python.org/issue34155
(cherry picked from commit 8cb65d1)

Co-authored-by: jpic [email protected]

https://bugs.python.org/issue34155

Before: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='a', domain='malicious.org'),) >>> parseaddr('[email protected]@important.com') ('', '[email protected]') After: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='', domain=''),) >>> parseaddr('[email protected]@important.com') ('', 'a@') https://bugs.python.org/issue34155 (cherry picked from commit 8cb65d1) Co-authored-by: jpic <[email protected]>
@miss-islington
Copy link
ContributorAuthor

@jpic: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
ContributorAuthor

@jpic: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
ContributorAuthor

@jpic: Status check is done, and it's a success ✅ .

Copy link
Contributor

@maxkingmaxking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@miss-islingtonmiss-islington merged commit 2170774 into python:3.8Aug 9, 2019
@miss-islingtonmiss-islington deleted the backport-8cb65d1-3.8 branch August 9, 2019 08:31
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-securityA security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@miss-islington@maxking@the-knights-who-say-ni@bedevere-bot@jpic