-
Notifications
You must be signed in to change notification settings - Fork 863
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
jackson 2.9.8 -> 2.9.9 #1269
jackson 2.9.8 -> 2.9.9 #1269
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1269 +/- ##
============================================
+ Coverage 70.12% 70.14% +0.01%
Complexity 188 188
============================================
Files 776 776
Lines 24122 24122
Branches 1799 1799
============================================
+ Hits 16916 16920 +4
+ Misses 6422 6420 -2
+ Partials 784 782 -2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes in Jackson 2.9.9 seems fine and don't have breaking changes. Approving the PR.
https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.9.9
There was a report of v2.9.9 introducing incompatible combination of versions. This might affect customers who use those dependencies. Holding off merging for further investigation. |
IMHO, this version bump should be prioritized as it addresses a security issue, CVE-2019-12086: FasterXML/jackson-databind#2326 |
@@ -0,0 +1,5 @@ | |||
{ | |||
"category": "core", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For core changes, please use 'AWS SDK for Java v2'
Perhaps bump it to 2.9.9.1 to address https://nvd.nist.gov/vuln/detail/CVE-2019-12814 |
obsolete |
…8a701de02 Pull request: release <- staging/e7f06a4f-475e-4797-969e-dd68a701de02
Description
Motivation and Context
Testing
Screenshots (if appropriate)
Types of changes
Checklist
mvn install
succeedsLicense