You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looks like the raising of GPXException inside the parse method uses a message about requiring a gpx root node, but it doesn't actually check for one. Instead, it is only checking for the existence of root. What this means in practice is that no errors are thrown if I load a different format's valid XML, for example a KML file.
I would like to propose that this line in parser.py be changed from:
ifrootisNone:
raisemod_gpx.GPXException('Document must have a `gpx` root node.')
to:
ifrootisNoneorroot.tag.lower() !='gpx':
raisemod_gpx.GPXException('Document must have a `gpx` root node.')
The text was updated successfully, but these errors were encountered:
Looks like the raising of GPXException inside the parse method uses a message about requiring a
gpx
root node, but it doesn't actually check for one. Instead, it is only checking for the existence of root. What this means in practice is that no errors are thrown if I load a different format's valid XML, for example a KML file.I would like to propose that this line in parser.py be changed from:
to:
The text was updated successfully, but these errors were encountered: