[proxy] github.com — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link
Contributor

miss-islington commented Aug 11, 2019 •
edited by bedevere-bot
Loading

This should fix the IndexError trying to retrieve DisplayName.display_name and DisplayName.value when the value is basically an empty string.

https://bugs.python.org/issue32178
(cherry picked from commit 09a1872)

Co-authored-by: Abhilash Raj maxking@users.noreply.github.com

https://bugs.python.org/issue32178

bpo-32178: Fix IndexError trying to parse 'To' header starting with '…
5d2372c 
…:'. (pythonGH-15044)

This should fix the IndexError trying to retrieve `DisplayName.display_name` and `DisplayName.value` when the `value` is basically an empty string.

https://bugs.python.org/issue32178
(cherry picked from commit 09a1872)

Co-authored-by: Abhilash Raj <maxking@users.noreply.github.com>
miss-islington requested a review from a team as a code owner August 11, 2019 20:45
bedevere-bot mentioned this pull request Aug 11, 2019
Merged
maxking added the type-bug An unexpected behavior, bug, or error label Aug 11, 2019
Copy link
Contributor Author

miss-islington commented Aug 11, 2019

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

miss-islington merged commit 9500bbe into python:3.8 Aug 11, 2019
miss-islington deleted the backport-09a1872-3.8 branch August 11, 2019 21:04
Copy link
Contributor Author

miss-islington commented Aug 11, 2019

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

1 similar comment
Copy link
Contributor Author

miss-islington commented Aug 11, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

maxking maxking approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants