Skip to content

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Feb 21, 2022

Curly brackets were never allowed in namespace URIs
according to RFC 3986, and so-called namespace-validating
XML parsers have the right to reject them a invalid URIs.

libexpat >=2.4.5 has become strcter in that regard due to
related security issues; with ET.XML instantiating a
namespace-aware parser under the hood, this test has no
future in CPython.

References:

Also, test_minidom.py: Support Expat >=2.4.5
(cherry picked from commit 2cae938)

Co-authored-by: Sebastian Pipping [email protected]

https://bugs.python.org/issue46811

Curly brackets were never allowed in namespace URIs
according to RFC 3986, and so-called namespace-validating
XML parsers have the right to reject them a invalid URIs.

libexpat >=2.4.5 has become strcter in that regard due to
related security issues; with ET.XML instantiating a
namespace-aware parser under the hood, this test has no
future in CPython.

References:
- https://datatracker.ietf.org/doc/html/rfc3968
- https://www.w3.org/TR/xml-names/

Also, test_minidom.py: Support Expat >=2.4.5
(cherry picked from commit 2cae938)

Co-authored-by: Sebastian Pipping <[email protected]>
@miss-islington
Copy link
Contributor Author

@hartwork and @ambv: Status check is done, and it's a failure ❌ .

@ambv ambv force-pushed the backport-2cae938-3.8 branch 7 times, most recently from f293fbe to c201d7d Compare February 22, 2022 11:01
@ambv ambv force-pushed the backport-2cae938-3.8 branch from c201d7d to f86bc12 Compare February 22, 2022 11:03
@ambv ambv merged commit 438f8cd into python:3.8 Feb 22, 2022
@miss-islington miss-islington deleted the backport-2cae938-3.8 branch February 22, 2022 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants