Skip to content

[SPARK-30333][CORE][BUILD][BRANCH-2.4] Upgrade jackson-databind to 2.6.7.3 - #26986

Closed
sandeep-katta wants to merge 2 commits into
apache:branch-2.4from
sandeep-katta:jacksonUpgrade
Closed

[SPARK-30333][CORE][BUILD][BRANCH-2.4] Upgrade jackson-databind to 2.6.7.3#26986
sandeep-katta wants to merge 2 commits into
apache:branch-2.4from
sandeep-katta:jacksonUpgrade

Conversation

@sandeep-katta

@sandeep-kattasandeep-katta commented Dec 23, 2019

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Upgrade jackson-databind to 2.6.7.3 to following CVE

CVE-2018-14718 - CVE-2018-14721
FasterXML/jackson-databind#2097

CVE-2018-19360, CVE-2018-19361, CVE-2018-19362
FasterXML/jackson-databind#2186

tag: https://github.com/FasterXML/jackson-databind/commits/jackson-databind-2.6.7.3

Why are the changes needed?

CVE-2018-14718,CVE-2018-14719,CVE-2018-14720,CVE-2018-14721,CVE-2018-19360,CVE-2018-19361,CVE-2018-19362

Does this PR introduce any user-facing change?

No

How was this patch tested?

Existing UT

@sandeep-katta

Copy link
Copy Markdown
ContributorAuthor

@srowensrowen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might have to run dev/test-dependencies.sh --replace-manifest but looks OK otherwise

@maropu

Copy link
Copy Markdown
Member

ok to test

@SparkQA

Copy link
Copy Markdown

Test build #115673 has finished for PR 26986 at commit 25de967.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@maropumaropu changed the title [SPARK-30333][CORE] Upgrade jackson-databind to 2.6.7.3[SPARK-30333][CORE][BUILD] Upgrade jackson-databind to 2.6.7.3Dec 24, 2019
maropu pushed a commit that referenced this pull request Dec 24, 2019
### What changes were proposed in this pull request?
Upgrade jackson-databind to 2.6.7.3 to following CVE
CVE-2018-14718 - CVE-2018-14721FasterXML/jackson-databind#2097CVE-2018-19360, CVE-2018-19361, CVE-2018-19362FasterXML/jackson-databind#2186
tag: https://github.com/FasterXML/jackson-databind/commits/jackson-databind-2.6.7.3
### Why are the changes needed?
CVE-2018-14718,CVE-2018-14719,CVE-2018-14720,CVE-2018-14721,CVE-2018-19360,CVE-2018-19361,CVE-2018-19362
### Does this PR introduce any user-facing change?
No
### How was this patch tested?
Existing UT
Closes#26986 from sandeep-katta/jacksonUpgrade.
Authored-by: sandeep katta <sandeep.katta2007@gmail.com>
Signed-off-by: Takeshi Yamamuro <yamamuro@apache.org>
@maropu

maropu commented Dec 24, 2019

Copy link
Copy Markdown
Member

@sandeep-katta oh... Can you open another new pr for master? In this pr, the target is branch-2.4 and I couldn't cherry-pick it cleanly..

@sandeep-katta

Copy link
Copy Markdown
ContributorAuthor

@sandeep-katta oh... Can you open another new pr for master? In this pr, the target is branch-2.4 and I couldn't cherry-pick it cleanly..

Master is using 2.10.0, so not required to merge this to master

@maropu

Copy link
Copy Markdown
Member

ah... if so, can you add [BRANCH-2.4] in the title?

@sandeep-kattasandeep-katta changed the title [SPARK-30333][CORE][BUILD] Upgrade jackson-databind to 2.6.7.3[SPARK-30333][CORE][BUILD][BRANCH-2.4] Upgrade jackson-databind to 2.6.7.3Dec 24, 2019
@maropu

Copy link
Copy Markdown
Member

ok, Thanks! Merged to branch-2.4.

@maropumaropu closed this Dec 24, 2019
@dongjoon-hyun

dongjoon-hyun commented Jan 17, 2020

Copy link
Copy Markdown
Member

Hi, All. It seems that we missed sbt build.
It's added at #26417 before this PR. I'll make a follow-up.

dongjoon-hyun added a commit that referenced this pull request Jan 17, 2020
### What changes were proposed in this pull request?
This PR aims to update `SparkBuild.scala` as a follow-up of
[SPARK-30333 Upgrade jackson-databind to 2.6.7.3](#26986).
### Why are the changes needed?
Since SPARK-29781, we override SBT Jackson dependency like Maven.
### Does this PR introduce any user-facing change?
No.
### How was this patch tested?
Pass the Jenkins.
Closes#27256 from dongjoon-hyun/SPARK-30333.
Authored-by: Dongjoon Hyun <dhyun@apple.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@sandeep-katta@maropu@SparkQA@dongjoon-hyun@srowen@HyukjinKwon