mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-01 07:33:08 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

#1745761, #755670, #13357, #12629, #1200313: improve attribute handling in HTMLParser.

Browse source
This commit is contained in:
Ezio Melotti 2011-11-14 18:04:05 +02:00
parent 74592919d6
commit 0f1571ce7f
3 changed files with 87 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

20
Lib/HTMLParser.py
View file

@ -24,22 +24,23 @@ starttagopen = re.compile('<[a-zA-Z]')
piclose = re.compile('>') piclose = re.compile('>')
commentclose = re.compile(r'--\s*>') commentclose = re.compile(r'--\s*>')
tagfind = re.compile('[a-zA-Z][-.a-zA-Z0-9:_]*') tagfind = re.compile('[a-zA-Z][-.a-zA-Z0-9:_]*')
attrfind = re.compile( attrfind = re.compile(
r'\s*([a-zA-Z_][-.:a-zA-Z_0-9]*)(\s*=\s*' r'\s*((?<=[\'"\s])[^\s/>][^\s/=>]*)(\s*=+\s*'
r'(\'[^\']*\'|"[^"]*"|[^\s"\'=<>`]*))?') r'(\'[^\']*\'|"[^"]*"|(?![\'"])[^>\s]*))?')
locatestarttagend = re.compile(r""" locatestarttagend = re.compile(r"""
<[a-zA-Z][-.a-zA-Z0-9:_]* # tag name <[a-zA-Z][-.a-zA-Z0-9:_]* # tag name
(?:\s+ # whitespace before attribute name (?:\s+ # whitespace before attribute name
(?:[a-zA-Z_][-.:a-zA-Z0-9_]* # attribute name (?:(?<=['"\s])[^\s/>][^\s/=>]* # attribute name
(?:\s*=\s* # value indicator (?:\s*=+\s* # value indicator
(?:'[^']*' # LITA-enclosed value (?:'[^']*' # LITA-enclosed value
|\"[^\"]*\" # LIT-enclosed value |"[^"]*" # LIT-enclosed value
|[^'\">\s]+ # bare value |(?!['"])[^>\s]* # bare value
) )
)? )?\s*
) )*
)* )?
\s* # trailing whitespace \s* # trailing whitespace
""", re.VERBOSE) """, re.VERBOSE)
endendtag = re.compile('>') endendtag = re.compile('>')
@ -254,6 +255,7 @@ class HTMLParser(markupbase.ParserBase):
elif attrvalue[:1] == '\'' == attrvalue[-1:] or \ elif attrvalue[:1] == '\'' == attrvalue[-1:] or \
attrvalue[:1] == '"' == attrvalue[-1:]: attrvalue[:1] == '"' == attrvalue[-1:]:
attrvalue = attrvalue[1:-1] attrvalue = attrvalue[1:-1]
if attrvalue:
attrvalue = self.unescape(attrvalue) attrvalue = self.unescape(attrvalue)
attrs.append((attrname.lower(), attrvalue)) attrs.append((attrname.lower(), attrvalue))
k = m.end() k = m.end()

76
Lib/test/test_htmlparser.py
View file

@ -226,13 +226,11 @@ DOCTYPE html [
self._parse_error("<a<a>") self._parse_error("<a<a>")
self._parse_error("</a<a>") self._parse_error("</a<a>")
self._parse_error("<!") self._parse_error("<!")
self._parse_error("<a $>")
self._parse_error("<a") self._parse_error("<a")
self._parse_error("<a foo='bar'") self._parse_error("<a foo='bar'")
self._parse_error("<a foo='bar") self._parse_error("<a foo='bar")
self._parse_error("<a foo='>'") self._parse_error("<a foo='>'")
self._parse_error("<a foo='>") self._parse_error("<a foo='>")
self._parse_error("<a foo=>")
def test_declaration_junk_chars(self): def test_declaration_junk_chars(self):
self._parse_error("<!DOCTYPE foo $ >") self._parse_error("<!DOCTYPE foo $ >")
@ -352,13 +350,85 @@ class AttributesTestCase(TestCaseBase):
self._run_check( self._run_check(
"<a a.b='v' c:d=v e-f=v>", "<a a.b='v' c:d=v e-f=v>",
[("starttag", "a", [("a.b", "v"), ("c:d", "v"), ("e-f", "v")])]) [("starttag", "a", [("a.b", "v"), ("c:d", "v"), ("e-f", "v")])])
self._run_check(
"<a $><b $=%><c \=/>",
[("starttag", "a", [("$", None)]),
("starttag", "b", [("$", "%")]),
("starttag", "c", [("\\", "/")])])
def test_entityrefs_in_attributes(self): def test_entityrefs_in_attributes(self):
self._run_check( self._run_check(
"<html foo='&euro;&amp;&#97;&#x61;&unsupported;'>", "<html foo='&euro;&amp;&#97;&#x61;&unsupported;'>",
[("starttag", "html", [("foo", u"\u20AC&aa&unsupported;")])]) [("starttag", "html", [("foo", u"\u20AC&aa&unsupported;")])])
def test_entities_in_attribute_value(self):
# see #1200313
for entity in ['&', '&amp;', '&#38;', '&#x26;']:
self._run_check('<a href="%s">' % entity,
[("starttag", "a", [("href", "&")])])
self._run_check("<a href='%s'>" % entity,
[("starttag", "a", [("href", "&")])])
self._run_check("<a href=%s>" % entity,
[("starttag", "a", [("href", "&")])])
def test_malformed_attributes(self):
# see #13357
html = (
"<a href=test'style='color:red;bad1'>test - bad1</a>"
"<a href=test'+style='color:red;ba2'>test - bad2</a>"
"<a href=test'&nbsp;style='color:red;bad3'>test - bad3</a>"
"<a href = test'&nbsp;style='color:red;bad4' >test - bad4</a>"
)
expected = [
('starttag', 'a', [('href', "test'style='color:red;bad1'")]),
('data', 'test - bad1'), ('endtag', 'a'),
('starttag', 'a', [('href', "test'+style='color:red;ba2'")]),
('data', 'test - bad2'), ('endtag', 'a'),
('starttag', 'a', [('href', u"test'\xa0style='color:red;bad3'")]),
('data', 'test - bad3'), ('endtag', 'a'),
('starttag', 'a', [('href', u"test'\xa0style='color:red;bad4'")]),
('data', 'test - bad4'), ('endtag', 'a')
]
self._run_check(html, expected)
def test_malformed_adjacent_attributes(self):
# see #12629
self._run_check('<x><y z=""o"" /></x>',
[('starttag', 'x', []),
('startendtag', 'y', [('z', ''), ('o""', None)]),
('endtag', 'x')])
self._run_check('<x><y z="""" /></x>',
[('starttag', 'x', []),
('startendtag', 'y', [('z', ''), ('""', None)]),
('endtag', 'x')])
# see #755670 for the following 3 tests
def test_adjacent_attributes(self):
self._run_check('<a width="100%"cellspacing=0>',
[("starttag", "a",
[("width", "100%"), ("cellspacing","0")])])
self._run_check('<a id="foo"class="bar">',
[("starttag", "a",
[("id", "foo"), ("class","bar")])])
def test_missing_attribute_value(self):
self._run_check('<a v=>',
[("starttag", "a", [("v", "")])])
def test_javascript_attribute_value(self):
self._run_check("<a href=javascript:popup('/popup/help.html')>",
[("starttag", "a",
[("href", "javascript:popup('/popup/help.html')")])])
def test_end_tag_in_attribute_value(self):
# see #1745761
self._run_check("<a href='http://www.example.org/\">;'>spam</a>",
[("starttag", "a",
[("href", "http://www.example.org/\">;")]),
("data", "spam"), ("endtag", "a")])
def test_main(): def test_main():
test_support.run_unittest(HTMLParserTestCase, AttributesTestCase) test_support.run_unittest(HTMLParserTestCase, AttributesTestCase)

3
Misc/NEWS
View file

@ -76,6 +76,9 @@ Core and Builtins
Library Library
------- -------
- Issues #1745761, #755670, #13357, #12629, #1200313: HTMLParser now correctly
handles non-valid attributes, including adjacent and unquoted attributes.
- Issue #13193: Fix distutils.filelist.FileList under Windows. The - Issue #13193: Fix distutils.filelist.FileList under Windows. The
"recursive-include" directive now recognizes both legal path separators. "recursive-include" directive now recognizes both legal path separators.