Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing mime4j library #22799

Merged
merged 2 commits into from Feb 1, 2017

Conversation

Projects
None yet
3 participants
@dadoonet
Copy link
Member

commented Jan 26, 2017

In some cases (apparently with outlook files), mime4j library is needed.
We removed it in the past which can cause elasticsearch to crash when you are using ingest-attachment (and probably mapper-attachments as well in 2.x series) with a file which requires this library.

Backport of #22764 in 5.x branch
Applying same changes for mapper-attachments plugin (5.x branch)

dadoonet added some commits Jan 24, 2017

Add missing mime4j library
In some cases (apparently with outlook files), mime4j library is needed.
We removed it in the past which can cause elasticsearch to crash when you are using ingest-attachment (and probably mapper-attachments as well in 2.x series) with a file which requires this library.

Backport of #22764 in 5.x branch
Add missing mime4j library
Applying same changes as #22764 but for mapper-attachments plugin (5.x branch)
@dadoonet

This comment has been minimized.

Copy link
Member Author

commented Jan 26, 2017

@jasontedor I also applied changes for mapper-attachments. Can you check it please?

@jasontedor
Copy link
Member

left a comment

LGTM.

@dadoonet

This comment has been minimized.

Copy link
Member Author

commented Feb 1, 2017

Thanks @jasontedor !

@dadoonet dadoonet merged commit 3bbf3cf into elastic:5.x Feb 1, 2017

1 check passed

CLA Commit author is a member of Elasticsearch
Details

@dadoonet dadoonet deleted the dadoonet:pr/22764-mime4j-5x branch Feb 1, 2017

@dadoonet

This comment has been minimized.

Copy link
Member Author

commented Feb 1, 2017

Also backported to 5.2 branch with 0a8a7eb and 70e3c9f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.