Skip to content

Conversation

@maxking
Copy link
Contributor

@maxking maxking commented Dec 8, 2019

Fix a potential IndexError when passing an empty value to the message-id
parser. Instead, HeaderParseError should be raised.

https://bugs.python.org/issue38708

Fix a potential IndexError when passing an empty value to the message-id
parser. Instead, HeaderParseError should be raised.
@maxking maxking requested a review from a team as a code owner December 8, 2019 06:29
@maxking maxking added needs backport to 3.8 type-bug An unexpected behavior, bug, or error labels Dec 8, 2019
Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@maxking maxking merged commit 3ae4ea1 into python:master Dec 9, 2019
@maxking maxking deleted the bpo-38708 branch December 9, 2019 01:37
@miss-islington
Copy link
Contributor

Thanks @maxking for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 9, 2019
…ythonGH-17504)

Fix a potential IndexError when passing an empty value to the message-id
parser. Instead, HeaderParseError should be raised.
(cherry picked from commit 3ae4ea1)

Co-authored-by: Abhilash Raj <maxking@users.noreply.github.com>
@bedevere-bot
Copy link

GH-17515 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Dec 9, 2019
…H-17504)

Fix a potential IndexError when passing an empty value to the message-id
parser. Instead, HeaderParseError should be raised.
(cherry picked from commit 3ae4ea1)

Co-authored-by: Abhilash Raj <maxking@users.noreply.github.com>
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
…ythonGH-17504)

Fix a potential IndexError when passing an empty value to the message-id
parser. Instead, HeaderParseError should be raised.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants