Skip to content

[improve][build] Revert "Upgrade jackson version to 2.15.0 for CVE-2022-1471 (#20177)" - #20182

Merged
mattisonchao merged 1 commit into
apache:masterfrom
mattisonchao:revert
Apr 26, 2023
Merged

[improve][build] Revert "Upgrade jackson version to 2.15.0 for CVE-2022-1471 (#20177)"#20182
mattisonchao merged 1 commit into
apache:masterfrom
mattisonchao:revert

Conversation

@mattisonchao

@mattisonchao mattisonchao commented Apr 26, 2023

Copy link
Copy Markdown
Member

This reverts commit fd60e9e.

Motivation

There's a regression of Jackson.
FasterXML/jackson-databind#3874

Modifications

  • Revert upgrading operation
  • doc-not-needed

Verifying this change

  • Make sure that the change passes the CI checks.

@github-actions

Copy link
Copy Markdown

@mattisonchao Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

@mattisonchao mattisonchao added this to the 3.1.0 milestone Apr 26, 2023
@github-actions github-actions Bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Apr 26, 2023
@mattisonchao mattisonchao reopened this Apr 26, 2023
@Technoboy- Technoboy- changed the title Revert "[improve][build] Upgrade jackson version to 2.15.0 for CVE-2022-1471 (#20177)" [improve][build] Revert "Upgrade jackson version to 2.15.0 for CVE-2022-1471 (#20177)" Apr 26, 2023
@codecov-commenter

codecov-commenter commented Apr 26, 2023

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.98%. Comparing base (fd60e9e) to head (db13ac0).
⚠️ Report is 2578 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #20182       +/-   ##
=============================================
+ Coverage     33.11%   72.98%   +39.86%     
- Complexity    12223    31976    +19753     
=============================================
  Files          1499     1868      +369     
  Lines        114461   138468    +24007     
  Branches      12429    15236     +2807     
=============================================
+ Hits          37906   101062    +63156     
+ Misses        71618    29374    -42244     
- Partials       4937     8032     +3095     
Flag Coverage Δ
inttests 24.14% <100.00%> (?)
systests 24.74% <100.00%> (?)
unittests 72.28% <100.00%> (+39.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ava/org/apache/pulsar/common/util/FieldParser.java 85.95% <100.00%> (+57.85%) ⬆️

... and 1540 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants