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

ZOOKEEPER-4429: Update jackson-databind to 2.13.1 #1788

Conversation

frederiko
Copy link
Contributor

This PR updates jackson-databind to 2.13.1 to address a raised vulnerability that could possible DoS attack certain versions of Jackson. Please refer to GH issue #3328 for further info. On top of that, it also fixes now deprecated PropertyNamingStrategy class initialization issue #2715.

@maoling
Copy link
Member

maoling commented Dec 24, 2021

@frederiko LGTM. I check this PR was applied to master is also OK. Please fix the master branch firstly next time.

@maoling
Copy link
Member

maoling commented Jan 11, 2022

@eolivelli PTAL

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

lgtm

@maoling
Copy link
Member

maoling commented Jan 17, 2022

If no other concerns, I'll merge it at this weekend(01-22).

asfgit pushed a commit that referenced this pull request Jan 22, 2022
This PR updates jackson-databind to 2.13.1 to address a raised vulnerability that could possible DoS attack certain versions of Jackson. Please refer to GH issue #3328 for further info. On top of that, it also fixes now deprecated PropertyNamingStrategy class initialization issue #2715.

Author: Frederiko Costa <frederiko@gmail.com>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Shoothzj <shoothzj@gmail.com>, maoling <maoling@apache.org>

Closes #1788 from frederiko/ZOOKEEPER-4414_jackson_databind_update
@maoling
Copy link
Member

maoling commented Jan 22, 2022

@frederiko
Thanks for your contribution.

@maoling maoling closed this Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants