James Bailey <james.bai...@gmail.com> added the comment:
Agreed with ruffsl's concerns about the overly aggressive detection of infinite
recursion.
I also wonder if the hrefs should be normalized or canonized for the check? The
check may miss infinite recursions if the hrefs
New submission from James Bailey:
After xml.etree.ElementInclude.include inserts an Xinclude'd href it does not
walk the just-inserted subtree to see if it contains any Xincludes itself.
I think the behaviour should be modified to walk the included subtree and
perform any Xincludes contained
