Email parse IndexError <""@wiarcom.com>

[ghost]

BPO	27931
Nosy	@warsaw, @bitdancer, @zhangyangyu, @Mariatta, @jayyyin
PRs	bpo-27931: Fix email address header parsing error #5329 [3.6] bpo-27931: Fix email address header parsing error (GH-5329) #5431
Files	issue27931.patch issue27931.patch issue27931_v2.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/bitdancer'
closed_at = <Date 2018-01-30.01:10:30.380>
created_at = <Date 2016-09-01.15:29:39.045>
labels = ['type-bug', '3.7', 'expert-email']
title = 'Email parse IndexError <""@wiarcom.com>'
updated_at = <Date 2018-01-30.01:10:30.379>
user = None

bugs.python.org fields:

activity = <Date 2018-01-30.01:10:30.379>
actor = 'xiang.zhang'
assignee = 'r.david.murray'
closed = True
closed_date = <Date 2018-01-30.01:10:30.380>
closer = 'xiang.zhang'
components = ['email']
creation = <Date 2016-09-01.15:29:39.045>
creator = '\xd0\x9a\xd0\xbe\xd0\xbd\xd1\x81\xd1\x82\xd0\xb0\xd0\xbd\xd1\x82\xd0\xb8\xd0\xbd \xd0\x92\xd0\xbe\xd0\xbb\xd0\xba\xd0\xbe\xd0\xb2'
dependencies = []
files = ['44340', '44516', '44523']
hgrepos = []
issue_num = 27931
keywords = ['patch']
message_count = 13.0
messages = ['274140', '274207', '274245', '274260', '275238', '275513', '275572', '278839', '285042', '307328', '310761', '311154', '311187']
nosy_count = 6.0
nosy_names = ['barry', 'r.david.murray', 'xiang.zhang', '\xd0\x9a\xd0\xbe\xd0\xbd\xd1\x81\xd1\x82\xd0\xb0\xd0\xbd\xd1\x82\xd0\xb8\xd0\xbd \xd0\x92\xd0\xbe\xd0\xbb\xd0\xba\xd0\xbe\xd0\xb2', 'Mariatta', 'jayyin11043']
pr_nums = ['5329', '5431']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue27931'
versions = ['Python 3.6', 'Python 3.7']

[ghost]

Email lib fails to parse some emails:

from email._header_value_parser import get_angle_addr    get_angle_addr('<""@wiarcom.com> SIZE=28113').addr_spec

IndexError: list index out of range

Seems that email address can be parsed.

[zhangyangyu]

Looking at the spec this is a valid angle addr. Not only et_angle_addr('<""@wiarcom.com> SIZE=28113')[0].addr_spec but also et_angle_addr('<""@wiarcom.com> SIZE=28113')[0].local_part fails for the same reason.

The problem exists in get_bare_quoted_string. When encounter bare '""', it returns a empty BareQuoteString. I write bpo-27931.patch to fix this.

[bitdancer]

You are correct, it is technically a valid angleaddr. I'll review this next week, the patch looks good (thanks) but I need to double check a couple things before I commit.

[zhangyangyu]

It should be. If there is anything wrong I'd like to hear and fix.

[zhangyangyu]

Ping this and hope you don't forget about it David. :)

[bitdancer]

Reviewed. There is something else that needs fixing. The address needs to re-render as ""@wiarcom.com, which it doesn't currently. I scanned the rendering code in _header_value_registry quickly and its not obvious to me why it doesn't work, so it may be a bug in the headerregistry.Address object. I'm uploading the patch with the added test.

[zhangyangyu]

You are right David. But I don't understand what it has to do with headerregistry. I update my original patch to get that behaviour, limit it the affection in angleaddr. Now seems everything is fine.

>>> email._header_value_parser.get_angle_addr('<""@wiarcom.com> SIZE=28113')[0].local_part
''
>>> email._header_value_parser.get_angle_addr('<""@wiarcom.com> SIZE=28113')[0].domain
'wiarcom.com'
>>> email._header_value_parser.get_angle_addr('<""@wiarcom.com> SIZE=28113')[0].route
>>> email._header_value_parser.get_angle_addr('<""@wiarcom.com> SIZE=28113')[0].addr_spec
'""@wiarcom.com'

[zhangyangyu]

Ping.

[ghost]

ping

[ghost]

ping

[jayyyin]

#5329
this is the pending pull request for this