[ https://issues.apache.org/jira/browse/TIKA-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17731752#comment-17731752 ]
Hudson commented on TIKA-4062: ------------------------------ SUCCESS: Integrated in Jenkins build Tika » tika-main-jdk11 #1111 (See [https://ci-builds.apache.org/job/Tika/job/tika-main-jdk11/1111/]) TIKA-4062: Test cases for custom error handling during XML parsing (#1186) (github: [https://github.com/apache/tika/commit/051ea7a3eb4125c64811ecb8db1d689b57082df0]) * (add) tika-core/src/test/java/org/apache/tika/sax/NonValidatingContentHandler.java * (add) tika-core/src/test/java/org/apache/tika/sax/ErrorResistantSAXParserFactory.java * (edit) tika-core/pom.xml * (add) tika-core/src/test/java/org/apache/tika/sax/CustomErrorHandlerTest.java * (add) tika-core/src/test/resources/test-documents/undeclared_entity.xml > OfflineContentHandler/ContentHandlerDecorator does not provide option for > custom error handling > ----------------------------------------------------------------------------------------------- > > Key: TIKA-4062 > URL: https://issues.apache.org/jira/browse/TIKA-4062 > Project: Tika > Issue Type: Bug > Components: tika-core > Affects Versions: 2.3.0, 2.4.0, 2.5.0, 2.6.0, 2.7.0, 2.8.0 > Reporter: Ravi Ranjan Jha > Priority: Critical > Fix For: 2.8.1 > > > OfflineContentHandler/ContentHandlerDecorator does not provide option for > custom error handling > Prior to the change of passing OfflineContentHandler to SAX Parser in > XMLReaderUtils.parseSAX, one could pass a custom ContentHandlerDecorator to > handle exception or override error/warning etc methods. The same is not > possible now because the default impl for handleException in the > OfflineContentHandler's parent ContentHandlerDecorator just throws exception > as shown below: > > protected void handleException(SAXException exception) throws SAXException { > throw exception; > } > > which could probably be (at minimum) > public void handleException(SAXException exception) throws SAXException { > handler.handleException(exception); > } > > This is breaking our app's behavior. Please take it as priority. -- This message was sent by Atlassian Jira (v8.20.10#820010)