Uploaded image for project: 'Tika'
  1. Tika
  2. TIKA-1673

Don't include source file name in embedded path with RecursiveParserWrapper

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      The RecursiveParserWrapper has been including the source file in the embedded file path ("X-TIKA:embedded_resource_path"), as in "test_recursive.docx/embed1.zip/embed2.zip/embed3.zip/embed3.txt". If the client forgets to send in a file name or if a filename doesn't exist, then the RecursiveParserWrapper defaults to "embed-1", which is wrong.

      Let's drop the source file name from the path so that the above will be "/embed1.zip/embed2.zip/embed3.zip/embed3.txt".

        Activity

        Hide
        tallison@mitre.org Tim Allison added a comment -

        r1688828

        Show
        tallison@mitre.org Tim Allison added a comment - r1688828
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in tika-trunk-jdk1.7 #779 (See https://builds.apache.org/job/tika-trunk-jdk1.7/779/)
        TIKA-1673 – doh, add back dropped qmino in server's pom (tallison: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1688828)

        • /tika/trunk/tika-server/pom.xml
          TIKA-1673 drop source file name from embedded file path; made a few java 7 updates; added timing for embedded docs (tallison: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1688827)
        • /tika/trunk/CHANGES.txt
        • /tika/trunk/tika-app/src/test/java/org/apache/tika/cli/TikaCLITest.java
        • /tika/trunk/tika-core/src/main/java/org/apache/tika/parser/RecursiveParserWrapper.java
        • /tika/trunk/tika-example/src/test/java/org/apache/tika/example/TestParsingExample.java
        • /tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/RecursiveParserWrapperTest.java
        • /tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/jdbc/SQLite3ParserTest.java
        • /tika/trunk/tika-server/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in tika-trunk-jdk1.7 #779 (See https://builds.apache.org/job/tika-trunk-jdk1.7/779/ ) TIKA-1673 – doh, add back dropped qmino in server's pom (tallison: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1688828 ) /tika/trunk/tika-server/pom.xml TIKA-1673 drop source file name from embedded file path; made a few java 7 updates; added timing for embedded docs (tallison: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1688827 ) /tika/trunk/CHANGES.txt /tika/trunk/tika-app/src/test/java/org/apache/tika/cli/TikaCLITest.java /tika/trunk/tika-core/src/main/java/org/apache/tika/parser/RecursiveParserWrapper.java /tika/trunk/tika-example/src/test/java/org/apache/tika/example/TestParsingExample.java /tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/RecursiveParserWrapperTest.java /tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/jdbc/SQLite3ParserTest.java /tika/trunk/tika-server/pom.xml

          People

          • Assignee:
            tallison@mitre.org Tim Allison
            Reporter:
            tallison@mitre.org Tim Allison
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development