[ 
https://issues.apache.org/jira/browse/TIKA-950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13407903#comment-13407903
 ] 

Nick Burch commented on TIKA-950:
---------------------------------

We don't want to pass in an InputStream if we can avoid it, as that causes the 
whole file to be buffered into memory

Any chance you could write a unit test that shows the problem? Only I can't 
quite understand what's going wrong from your description :( With a unit test, 
we can try some other options out, and hopefully find a solution that doesn't 
need to buffer the whole file into memory
                
> Wrong Office Open XML detection in ZipContainerDetector
> -------------------------------------------------------
>
>                 Key: TIKA-950
>                 URL: https://issues.apache.org/jira/browse/TIKA-950
>             Project: Tika
>          Issue Type: Bug
>          Components: mime
>            Reporter: Marco Quaranta
>            Priority: Minor
>              Labels: detection, ooxml
>             Fix For: 1.1
>
>         Attachments: ZipContainerDetector.diff
>
>
> Method detectOfficeOpenXML() in ZipContainerDetector class does not detect 
> correctly an ooxml file due to an Exception throwed by OPCPackage.open(..) 
> POI's class. This class make use of ZipFile when its method is called with 
> (as Tika do) filePath string and in this way it generates an exception; 
> passing instead a fileInputStream makes POI correctly detects OfficeOpenXML 
> formats.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to