Skip to content
This repository has been archived by the owner on Oct 27, 2021. It is now read-only.

Property 'http://javax.xml.XMLConstants/property/accessExternalDTD #432

Merged
merged 2 commits into from
Feb 5, 2021

Conversation

dixonwhitmire
Copy link
Member

@dixonwhitmire dixonwhitmire commented Feb 4, 2021

This PR resolves an issue where an Apache Tika transitive dependency, xerces, generates the following stack trace and warning

org.xml.sax.SAXNotRecognizedException: Property 'http://javax.xml.XMLConstants/property/accessExternalDTD' is not recognized.
        at org.apache.xerces.jaxp.validation.XMLSchemaFactory.setProperty(Unknown Source)
        at org.apache.camel.support.processor.validation.SchemaReader.createSchemaFactory(SchemaReader.java:184)
        at org.apache.camel.support.processor.validation.SchemaReader.getSchemaFactory(SchemaReader.java:157)
        at org.apache.camel.support.processor.validation.SchemaReader.createSchema(SchemaReader.java:198)
        at org.apache.camel.support.processor.validation.SchemaReader.loadSchema(SchemaReader.java:89)
        at org.apache.camel.component.validator.ValidatorEndpoint.createProducer(ValidatorEndpoint.java:121)

https://bugs.openjdk.java.net/browse/JDK-8200177
https://stackoverflow.com/questions/58374278/org-xml-sax-saxnotrecognizedexception-property-http-javax-xml-xmlconstants-p

Removing the xerces implementation allows the application to utilize the appropriate implementations that are now included with the JRE/JDK.

I also updated LFH connect's camel dependency from 3.4 to 3.7.1.

Note: I was able to determine which library "pulled' in xerces using the ./gradlew dependencies command.

resolves #418

@dixonwhitmire dixonwhitmire self-assigned this Feb 4, 2021
@dixonwhitmire dixonwhitmire requested a review from ccorley February 4, 2021 03:06
@dixonwhitmire dixonwhitmire merged commit 4ddd6c4 into master Feb 5, 2021
@dixonwhitmire dixonwhitmire deleted the dtd-error branch February 5, 2021 18:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DTD Warning on Application Startup
2 participants