-
Notifications
You must be signed in to change notification settings - Fork 357
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
Property 'http://javax.xml.XMLConstants/property/accessExternalDTD' is not recognized.' #154
Comments
Regards to this issue, openhtmltopdf worked flawlessly when executed only with JSE8, but failed when executed with WildFly and Arquillian in a JEE7 application. |
Same case here. After patching XMLResource.java for bypassing #54, I'm now hitting this. Been a rough testride so far under JBoss containers :/ WildFly 10.1 running on OpenJDK 8 (131-b11) on Debian Jessie. |
Pull Request #155 had been submitted to fix this issue. Waiting for merge. |
… working on JBoss, Wildfly. Thanks @estevandiedrich
This should be fixed by above commit. I'll close this issue and we can continue JBoss discussion in #54 (hopefully it should all work now). Thanks everyone. |
@danfickle in this solution you are swallowing the exception every time it throws. Will it help in preventing the xxe attacks. I saw a similar bug at ModelDriven/fUML-Reference-Implementation#26 which might be a cause for this issue too |
Hi everybody, we are trying to use openhtml2pdf and we had some problems with xalan. After some bug we found where the exception has been thrown, and it was in 'openhtmltopdf-core/src/main;java/com/openhtmltopdf/resource/XMLResource.java' in line 222 aproximadetly, we just add a try/catch to ignore when the attribute XMLConstants.ACCESS_EXTERNAL_DTD is not supported by the xalan. Probably its not the best solucion for this problem, but its a workaround that solves the problem.
The text was updated successfully, but these errors were encountered: