Skip to content

fix: upgrade jackson-databind to 2.21.1#319

Merged
i-am-SR merged 1 commit intov2from
jackson
Mar 12, 2026
Merged

fix: upgrade jackson-databind to 2.21.1#319
i-am-SR merged 1 commit intov2from
jackson

Conversation

@i-am-SR
Copy link
Copy Markdown
Contributor

@i-am-SR i-am-SR commented Mar 12, 2026

Description

Why is this change being made?

  1. Fix for CVE

What is changing?

  1. Upgrade jackson-databind to 2.21.1

Related Links

  • Issue #, if available: N/A

Testing

How was this tested?

  1. Local build and unit tests with mvn clean test install.
  2. Tested with Java App that was able to connect to RDS PG DB.

When testing locally, provide testing artifact(s):

>  mvn clean test install
.
.
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 133, Failures: 0, Errors: 0, Skipped: 0
.
.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  10.275 s
[INFO] Finished at: 2026-03-12T14:49:52-07:00
[INFO] ------------------------------------------------------------------------

Reviewee Checklist

Update the checklist after submitting the PR

  • I have reviewed, tested and understand all changes
    If not, why:
  • I have filled out the Description and Testing sections above
    If not, why:
  • Build and Unit tests are passing
    If not, why:
  • Unit test coverage check is passing
    If not, why:
  • I have ensured no sensitive information is leaking (i.e., no logging of sensitive fields, or otherwise)
    If not, why:
  • I have added explanatory comments for complex logic, new classes/methods and new tests
    If not, why: Just a version upgrade. No changes.
  • I have updated README/documentation (if needed)
    If not, why: Just a version upgrade. No changes.
  • I have clearly called out breaking changes (if any)
    If not, why: Just a minor version upgrade. No changes.

Reviewer Checklist

All reviewers please ensure the following are true before reviewing:

  • Reviewee checklist has been accurately filled out
  • Code changes align with stated purpose in description
  • Test coverage adequately validates the changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@i-am-SR i-am-SR requested a review from a team as a code owner March 12, 2026 22:07
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.24%. Comparing base (9bd96fc) to head (c4b22b2).
⚠️ Report is 1 commits behind head on v2.

Additional details and impacted files
@@            Coverage Diff            @@
##                 v2     #319   +/-   ##
=========================================
  Coverage     82.24%   82.24%           
  Complexity      143      143           
=========================================
  Files            12       12           
  Lines           366      366           
  Branches         45       45           
=========================================
  Hits            301      301           
  Misses           58       58           
  Partials          7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@i-am-SR i-am-SR merged commit 16539a8 into v2 Mar 12, 2026
5 checks passed
@i-am-SR i-am-SR deleted the jackson branch March 12, 2026 22:16
Sign up for free to 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.

3 participants