HtmlParser non-strict goes wrong with unquoted attributes

[svilend]

BPO	12008
Nosy	@ezio-melotti, @merwok, @bitdancer
Files	html.parser.diff: patch to limit nonstruct regexp's span test-htmlparser-attrs.py: standalone test

^{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/ezio-melotti'
closed_at = <Date 2011-11-01.12:46:40.385>
created_at = <Date 2011-05-05.10:47:27.403>
labels = ['type-bug', 'library']
title = 'HtmlParser non-strict goes wrong with unquoted attributes'
updated_at = <Date 2011-11-01.12:46:40.351>
user = 'https://bugs.python.org/svilend'

bugs.python.org fields:

activity = <Date 2011-11-01.12:46:40.351>
actor = 'ezio.melotti'
assignee = 'ezio.melotti'
closed = True
closed_date = <Date 2011-11-01.12:46:40.385>
closer = 'ezio.melotti'
components = ['Library (Lib)']
creation = <Date 2011-05-05.10:47:27.403>
creator = 'svilend'
dependencies = []
files = ['21893', '21894']
hgrepos = []
issue_num = 12008
keywords = ['patch']
message_count = 5.0
messages = ['135182', '135183', '143472', '146772', '146773']
nosy_count = 6.0
nosy_names = ['ezio.melotti', 'eric.araujo', 'r.david.murray', 'pietvo', 'python-dev', 'svilend']
pr_nums = []
priority = 'normal'
resolution = 'out of date'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue12008'
versions = ['Python 3.2', 'Python 3.3']

[svilend]

nonstrict mode seems to eat too much into data and gets past endpos of the chunk processed, and parser gets confused and treats any subsequent stuff as data. i didn't think out how to fix the regexp as such, but instead limited its span to :endpos so it doesnot eat too much.
seems to happen with unquoted attributes.

[svilend]

(the nonstrict regexp came with bpo-1046092)

[pietvo]

I was bitten by this bug today. Hope it will be solved in the next release of Python 3.

It is also possible to use the third argument of search in line 285:

                m = attrfind_tolerant.search(rawdata, k, endpos)

This seems to me to be a more `natural' solution.

[python-dev]

New changeset 6107a84e3c44 by Ezio Melotti in branch '3.2':
bpo-12008: add a test.
http://hg.python.org/cpython/rev/6107a84e3c44

New changeset 495b31a8b280 by Ezio Melotti in branch 'default':
bpo-12008: merge with 3.2.
http://hg.python.org/cpython/rev/495b31a8b280

[ezio-melotti]

This seems to be already fixed in 3.2/3.3, so I extracted the test from your script and added to the test suite. If you can find a way to break the parser let me know.