Skip to content

Jackson version bump to 2.18.9 - #635

Open
rudraroop wants to merge 3 commits into
mainfrom
rudraroop/jackson-bump
Open

Jackson version bump to 2.18.9#635
rudraroop wants to merge 3 commits into
mainfrom
rudraroop/jackson-bump

Conversation

@rudraroop

Copy link
Copy Markdown

Issue #, if available:

Description of changes:
Upgrade jackson dependency version to 2.18.9 in the serialization package to address the following CVE reports CVE-2026-54512 and CVE-2026-54513

Will update tests package separately after maven workflow's auto version bump for serialization

Target (OCI, Managed Runtime, both):

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

@rudraroop
rudraroop requested review from darklight3it, fabisev and maxday and a lite review from Copilot August 19, 2026 16:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

This PR upgrades the Jackson dependency version used by aws-lambda-java-serialization to address reported CVEs, and bumps the tests module to consume the new serialization version.

Changes:

  • Bump aws-lambda-java-serialization Jackson version from 2.18.6 to 2.18.9.
  • Update aws-lambda-java-tests to depend on aws-lambda-java-serialization 1.4.2.
  • Add a 1.4.2 entry to the serialization release changelog.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
aws-lambda-java-tests/pom.xml Updates tests module to use serialization 1.4.2.
aws-lambda-java-serialization/pom.xml Bumps Jackson version property to 2.18.9.
aws-lambda-java-serialization/RELEASE.CHANGELOG.md Documents the 1.4.2 release and dependency bump.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread aws-lambda-java-serialization/RELEASE.CHANGELOG.md
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.38%. Comparing base (b668333) to head (9bd304e).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #635   +/-   ##
=========================================
  Coverage     65.38%   65.38%           
  Complexity      212      212           
=========================================
  Files            34       34           
  Lines           991      991           
  Branches        143      143           
=========================================
  Hits            648      648           
  Misses          290      290           
  Partials         53       53           

☔ View full report in Codecov by Harness.
📢 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.

-->
<junit.version>5.9.2</junit.version>
<jacoco.maven.plugin.version>0.8.7</jacoco.maven.plugin.version>
<aws-lambda-java-serialization.version>1.4.1</aws-lambda-java-serialization.version>

@darklight3it darklight3it Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It's fine but we need to remember of releasing also this package in the followup PR.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

actually I was wrong. The test are exercised locally but you need to have the same version, otherwise it will fail the test, because it is registered locally as 1.4.1.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Cool, I'll revert this then

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