You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/01/27 20:50:00 UTC
[jira] [Commented] (TIKA-3418) DefaultZipContainerDetector does not support loading of ZipContainerDetectors in an OSGi enviroment
[ https://issues.apache.org/jira/browse/TIKA-3418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17483429#comment-17483429 ]
ASF GitHub Bot commented on TIKA-3418:
--------------------------------------
tballison commented on pull request #446:
URL: https://github.com/apache/tika/pull/446#issuecomment-1023626019
I merged this and made a few modifications. I need to figure out how to do a better job of passing tikaconfig info into DefaultZipDetector.
Thank you @simon-shlomo for opening this!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
> DefaultZipContainerDetector does not support loading of ZipContainerDetectors in an OSGi enviroment
> ---------------------------------------------------------------------------------------------------
>
> Key: TIKA-3418
> URL: https://issues.apache.org/jira/browse/TIKA-3418
> Project: Tika
> Issue Type: Improvement
> Components: core, parser
> Affects Versions: 2.0.0
> Reporter: Simon-Shlomo Poil
> Priority: Major
>
> Classes that implements the ZipContainerDetector interface are loaded via the ServiceLoader in the DefaultZipContainerDetector (in parsers-zip-commons).
> In an OSGi enviroment the current behaviour is (default constructor) to only load classes that a found by the DefaultZipContainerDetector classloader, however this will miss ZipContainerDetector classes such as the [OPCPackageDetector.java|https://github.com/apache/tika/blob/main/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java] in the [tika-parser-microsoft-module|https://github.com/apache/tika/tree/main/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module].
> The DefaultZipConatinerDetector has constructor to pass a ServiceLoader, which in principle could allow loading of dynamic services. However, this constructor will load the services only on initialization, which does comply with the dynamic natur of an OSGi enviroment where services can come and go dynamically.
> Linked to this issue is pull-request that attempts solve this issue following a similar strategy of service loading as implemented in the DefaultDetector and DefaultParser.
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)