[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Irit Katriel : -- versions: +Python 3.10, Python 3.11, Python 3.9 -Python 3.6, Python 3.7, Python 3.8 ___ Python tracker ___

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Yassir Karroum : -- nosy: +ukarroum nosy_count: 7.0 -> 8.0 pull_requests: +28814 pull_request: https://github.com/python/cpython/pull/30612 ___ Python tracker ___

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Cheryl Sabella : -- nosy: +scoder ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Jonathan Gossage : -- pull_requests: +9632 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Jonathan Gossage added the comment: The other thing to consider which also supports option 2 is that xml.parsers.expat provides an interface to the Expat parser which is easier to use and more complete than the Sax parser implementation and is the implementation likely to be used by anyone

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Tal Einat added the comment: See additional research and discussion in the comments of PR GH-9715. Simply changing this to return a string rather than bytes would break backwards compatibility. I certainly agree that this should have returned a string in the first place, especially since

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Jonathan Gossage : -- pull_requests: +9101 stage: test needed -> patch review ___ Python tracker ___ ___ Python-bugs-list

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Change by Zachary Ware : -- nosy: +christian.heimes ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Amaury Forgeot d'Arc amaur...@gmail.com added the comment: A unit test (or even a sample script) showing the desired feature is needed. -- nosy: +amaury.forgeotdarc stage: - unit test needed ___ Python tracker rep...@bugs.python.org

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Martin v. Löwis mar...@v.loewis.de added the comment: Would you like to contribute a patch? -- nosy: +loewis ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6686 ___

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Colin Stewart co...@owlfish.com added the comment: I'm not familiar with the inner workings of the expat integration with Python, so the attached patches need careful review. The first patch (expatreader.py.patch) is the minimum to resolve this issue. The second patch (expatreader.py.patch2)

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

Colin Stewart co...@owlfish.com added the comment: Adding second patch. -- Added file: http://bugs.python.org/file14702/expatreader.py.patch2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6686

[issue6686] xml.sax.xmlreader.XMLReader.getProperty (xml.sax.handler.property_xml_string) returns bytes

New submission from Colin Stewart co...@owlfish.com: The documentation for the xml.sax.handler.property_xml_string SAX property states that it should be data type: String. However when retrieving this value in Python 3.1 it returns a bytes object instead. This makes handling the returned value