This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Allow more low-level parser configuration in ElementTree
Type: behavior Stage:
Components: XML Versions: Python 3.10
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: brechtm, eli.bendersky, r.david.murray, scoder
Priority: normal Keywords:

Created on 2013-11-03 10:58 by brechtm, last changed 2022-04-11 14:57 by admin.

Files
File name Uploaded Description Edit
Screenshot_20200223-214519.png Hammad, 2020-02-23 16:45
Messages (6)
msg202011 - (view) Author: Brecht Machiels (brechtm) Date: 2013-11-03 10:58
With Python 3.2, I subclassed ElementTree.XMLParser to set ExternalEntityRefHandler on the XMLParser's (expat) 'parser' member. I understand the 'parser' member is not part of the public API, but this was the only way to customize the parser without having to write a parser from scratch.

With 3.3, cElementTree replaces the Python implementation by default. Its XMLParser class has no accessible 'parser' member to configure. Unfortunately, there does not seem to be a way to use the pure-Python XMLParser, which would still allow for customization of the parser. Why is the Python version still in the library if it can't be accessed? Only for platforms where the C extension is not available?

I see two possible solutions:

1) Have XMLParser (both the C and Python versions) accept an optional parser argument, so that a custom parser can be passed in.

2) Make the Python version of ElementTree available again.

My other option is to copy the Python XMLParser version into my project. I would like to avoid this, as this would duplicate a lot of perfectly good code.
Perhaps there are other solutions?
msg202034 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2013-11-03 16:26
It is there so that Python implementations (other than cPython) that do not have ElementTree accelerator modules can fall back on the pure python version.

You can import the pure python version in cPython by blocking the import of the C accelerator:

   import sys
   sys.modules['_elementtree'] = None
   import xml.etree.ElementTree

I'll leave this open to see what the xml experts think about the custom parser idea.
msg202037 - (view) Author: Stefan Behnel (scoder) * (Python committer) Date: 2013-11-03 16:43
Brecht Machiels, 03.11.2013 11:58:
> With Python 3.2, I subclassed ElementTree.XMLParser to set ExternalEntityRefHandler on the XMLParser's (expat) 'parser' member

This sounds like a request for a missing feature to me. Here is how lxml
handles it:

http://lxml.de/resolvers.html

> I understand the 'parser' member is not part of the public API

Correct, although the fact that it is not prefixed with an underscore is
rather unfortunate. IMHO, there is no guarantee that the underlying parser
will always be expat, and in fact, it is not in lxml.

> I see two possible solutions:
> 1) Have XMLParser (both the C and Python versions) accept an optional parser argument, so that a custom parser can be passed in.

-1. IMHO, the underlying parser should not be made a part of the API.
Instead, whatever features it provides that are needed on user side should
be mapped and abstracted in one way or another. Resolving external entity
references is something that should not be done as low-level as expat.
msg202040 - (view) Author: Brecht Machiels (brechtm) Date: 2013-11-03 17:23
> You can import the pure python version in cPython by blocking the import > of the C accelerator:
>
>   import sys
>   sys.modules['_elementtree'] = None
>   import xml.etree.ElementTree

This will not work if xml.etree.ElementTree was already imported by another module, I think. Of course, you can take care of that case, but it doesn't get any prettier (I have done this for now).

Any objections to making the pure Python versions available under a different name?
msg202049 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2013-11-03 18:30
Yes, you have to do the sys.modules set at the start of your application.

The python module is what gets imported, it is just that the C classes override some of the classes from the Python module when they are imported.  So no, there's no way to make the pure python available in cPython other than blocking the _elementtree import at application startup.

If you want to discuss changing this fact, it would be a broader discussion in the context of PEP 399, and should take place on the python-dev mailing list.
msg376546 - (view) Author: Stefan Behnel (scoder) * (Python committer) Date: 2020-09-08 05:01
Changing subject to make it clear what this ticket is really about.
History
Date User Action Args
2022-04-11 14:57:53adminsetgithub: 63682
2020-09-08 05:01:02scodersettitle: Pure-Python ElementTree classes no longer available since 3.3 -> Allow more low-level parser configuration in ElementTree
messages: + msg376546
versions: + Python 3.10, - Python 3.3, Python 3.4, Python 3.5
2020-02-23 16:45:56Hammadsetfiles: + Screenshot_20200223-214519.png
2013-11-03 18:30:49r.david.murraysetmessages: + msg202049
2013-11-03 17:23:34brechtmsetmessages: + msg202040
2013-11-03 16:43:48scodersetmessages: + msg202037
2013-11-03 16:26:38r.david.murraysetnosy: + r.david.murray
messages: + msg202034
2013-11-03 10:58:40brechtmcreate