Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upUpdate Apache Tika - security vulnerabilities; resolves #131. #285
Conversation
ruebot
requested review from
ianmilligan1 and
lintool
Oct 17, 2018
ruebot
referenced this pull request
Oct 17, 2018
Open
DetectLanguage.scala: class LanguageIdentifier in package language is deprecated #286
ianmilligan1
approved these changes
Oct 17, 2018
Looks good to me! Out of curiosity, tested the .keepLanguages
command and all still works.
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
phew |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
codecov-io
Oct 17, 2018
Codecov Report
Merging #285 into master will not change coverage.
The diff coverage isn/a
.
@@ Coverage Diff @@
## master #285 +/- ##
=======================================
Coverage 70.36% 70.36%
=======================================
Files 41 41
Lines 1046 1046
Branches 192 192
=======================================
Hits 736 736
Misses 244 244
Partials 66 66
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 0d4f9bb...1f27d8e. Read the comment docs.
codecov-io
commented
Oct 17, 2018
Codecov Report
@@ Coverage Diff @@
## master #285 +/- ##
=======================================
Coverage 70.36% 70.36%
=======================================
Files 41 41
Lines 1046 1046
Branches 192 192
=======================================
Hits 736 736
Misses 244 244
Partials 66 66 Continue to review full report at Codecov.
|
ruebot commentedOct 17, 2018
•
edited
GitHub issue(s): #131
What does this Pull Request do?
Update Apache Tika to 1.19.1 for the following security vulnerabilities:
How should this be tested?
TravisCI should take care of it. If there is any language analysis scripts laying around, we should double check those.
Additional Notes:
This creates a build warning:
Follow-on ticket #286