-
Notifications
You must be signed in to change notification settings - Fork 24.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove support for Visio and potm files
* Send a non supported document to an ingest pipeline using `ingest-attachment` * If Tika is not able to parse the document because of a missing class (we are not importing all jars needed by Tika), Tika throws a Throwable which is not catch. This commit removes support for Visio and POTM office files. So elasticsearch is not killed anymore when you run a command like: ``` GET _ingest/pipeline/_simulate { "pipeline" : { "processors" : [ { "attachment" : { "field" : "file" } } ] }, "docs" : [ { "_source" : { "file" : "BASE64CONTENT" } } ] } ``` The good news is that it does not kill the node anymore and allows to extract the text which is in the Office document even if we have a Visio content (which is not extracted anymore). Related to #22077 Backport of #23214 in 5.3 branch (cherry picked from commit 76a977a)
- Loading branch information
Showing
7 changed files
with
46 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file added
BIN
+97.5 KB
.../src/test/resources/org/elasticsearch/ingest/attachment/test/sample-files/issue-22077.doc
Binary file not shown.
Binary file added
BIN
+80.8 KB
...src/test/resources/org/elasticsearch/ingest/attachment/test/sample-files/issue-22077.docx
Binary file not shown.
Binary file added
BIN
+206 KB
...src/test/resources/org/elasticsearch/ingest/attachment/test/sample-files/issue-22077.vsdx
Binary file not shown.
Binary file removed
BIN
-30.8 KB
...t/src/test/resources/org/elasticsearch/ingest/attachment/test/tika-files/testPPT.potm.zip
Binary file not shown.