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.

Sign up for GitHub

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

Jump to bottom

[SPARK-29445][CORE] Bump netty-all from 4.1.39.Final to 4.1.42.Final #26099

Closed
wants to merge 1 commit into from
Closed

[SPARK-29445][CORE] Bump netty-all from 4.1.39.Final to 4.1.42.Final #26099

wants to merge 1 commit into from

Conversation

Fokko
Copy link
Contributor

@Fokko Fokko commented Oct 12, 2019

What changes were proposed in this pull request?

Minor version bump of Netty to patch reported CVE.

Patches: https://www.cvedetails.com/cve/CVE-2019-16869/

Why are the changes needed?

Does this PR introduce any user-facing change?

No

How was this patch tested?

Compiled locally using mvn clean install -DskipTests

@wangyum
Copy link
Member

wangyum commented Oct 12, 2019

ok to test

@SparkQA
Copy link

SparkQA commented Oct 12, 2019

Test build #111961 has finished for PR 26099 at commit 37cc5f7.

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

@srowen srowen closed this in b5b1b69 Oct 12, 2019
srowen pushed a commit that referenced this pull request Oct 12, 2019
@Fokko @srowen
[SPARK-29445][CORE] Bump netty-all from 4.1.39.Final to 4.1.42.Final
b2f96a5 
Minor version bump of Netty to patch reported CVE.

Patches: https://www.cvedetails.com/cve/CVE-2019-16869/

No

Compiled locally using `mvn clean install -DskipTests`

Closes #26099 from Fokko/SPARK-29445.

Authored-by: Fokko Driesprong <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit b5b1b69)
Signed-off-by: Sean Owen <[email protected]>
@srowen
Copy link
Member

srowen commented Oct 12, 2019

Merged to master/2.4

@Fokko Fokko deleted the SPARK-29445 branch October 12, 2019 17:40
@Fokko
Copy link
Contributor Author

Fokko commented Oct 12, 2019

Thanks Sean for the quick response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@srowen srowen srowen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Fokko @wangyum @SparkQA @srowen