[
https://issues.apache.org/jira/browse/TIKA-597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13004621#comment-13004621
]
Jukka Zitting commented on TIKA-597:
------------------------------------
I don't think the parse() method should ever intentionally throw
RuntimeExceptions. Thus in revision 1079857 I modified the code to use the
declared Mime- instead of RuntimeExceptions to wrap potential Tika- and
SAXExceptions.
I also added code to unwrap such exceptions where appropriate. This allows the
original exceptions to pass through the call chain even if only MimeExceptions
are declared for one segment of that chain.
> Bogus exception handler in
> org.apache.tika.parser.mail.MailContentHandler.body(BodyDescriptor,
> InputStream)
> -----------------------------------------------------------------------------------------------------------
>
> Key: TIKA-597
> URL: https://issues.apache.org/jira/browse/TIKA-597
> Project: Tika
> Issue Type: Bug
> Components: parser
> Affects Versions: 0.8
> Reporter: Benson Margulies
> Assignee: Julien Nioche
> Fix For: 1.0
>
> Attachments: TIKA-597.patch
>
>
> org.apache.tika.parser.mail.MailContentHandler.body(BodyDescriptor,
> InputStream)
> contains an exception handler that calls printStackTrace instead of rethrowing
> as a RuntimeException. Should it be 'throws TikaException' in any case?
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
