New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
XML::LibXML 2.0201+ requires enabling of External DTDs #511
Comments
Just adding this note - gathering the info in one place. From EP-Tech:
|
Other places that might need to be addressed: eprints/perl_lib/EPrints/XML/LibXML.pm Lines 71 to 74 in 2d42b2b
|
Hi guys, as this is of actuality again :) I was wondering few years ago, why EPrints is including these libraries as part of the code and it is not using a dependency system |
@denics it might be because, inolder versions, EPrints supported other XML libraries (GDOME, DOM), and the EPrints::XML::zzz modules were to create a more standard way to interface to all of them? I'd hope that somewhere in the 3.4 series, there will be a review of the code base, and refactor on a standard library (LibXML seems to be the winner!?). |
Yep, indeed you are right, but now we should be able to get rid of all of them. I do not remember where I did propose some patches, I do not think in here, maybe in the eprintsug group. I remember that I did that because exactly libXML that had a major critical bug and EP was still using the old one. It is not a big job to rely on system libs now Perl using carton or even cpanm. |
Reported by Paolo Tealdi via Eprints-tech
this version the module needs one more MANDATORY option at startup, that enables
external DTD parsing, now disabled by default.
The text was updated successfully, but these errors were encountered: