Repository navigation
[4.22.x] Upgrade Jackson 2 to 2.22.4 and Jackson 3 to 3.2.4 - #29712
Conversation
gnodet-bot
left a comment
There was a problem hiding this comment.
This is a security-critical dependency upgrade — 12 CVEs are referenced in the description, and the upgrade is a backport to camel-4.22.x of what is already applied on main (which already carries 2.22.4 / 3.2.4). The bump itself is correct and the target branch is appropriate.
CVE description accuracy: Cross-referencing the 12 CVEs listed against the upstream Jackson release notes (jackson-databind, jackson-core):
-
Confirmed in Jackson 2.22.1–2.22.4 or Jackson 3.2.2–3.2.4 release notes:
- CVE-2026-68497 (jackson-databind 2.22.2:
XMLGregorianCalendarnumber-len constraint) - CVE-2026-68498 (jackson-core 2.22.x / jackson-core 3.2.2:
maxNameLengthenforcement) - CVE-2026-89407 (jackson-core:
looksLikeValidNumberfix) - CVE-2026-89425 (jackson-core:
UTF8DataInputJsonParsertoken-length fix) - CVE-2026-91777 (jackson-databind 2.22.3: quadratic forward-reference resolution)
- CVE-2026-110879 (jackson-databind 2.22.4:
PolymorphicTypeValidatortype-parameter bypass)
- CVE-2026-68497 (jackson-databind 2.22.2:
-
Not found in the Jackson 2.22.x or 3.2.x release notes reviewed:
- CVE-2026-68495, CVE-2026-68496, CVE-2026-104016, CVE-2026-104895, CVE-2026-106554, CVE-2026-110747
These 6 CVEs may belong to other Jackson sub-projects (e.g.
jackson-dataformat-*,jackson-modules-*) or were generated incorrectly. Please verify each one maps to an actual CVE and that it is patched by this exact version bump — if any are from components not managed by thejackson2-versionorjackson3-versionBOM properties, the description should clarify that or omit the unrelated IDs. -
Also fixed but not listed in the description:
- CVE-2026-54515, CVE-2026-59889, CVE-2026-77310 (jackson-databind 2.22.1)
- CVE-2026-19032, CVE-2026-83557 (jackson-databind 2.22.2)
- CVE-2026-91776 (jackson-databind 2.22.3)
The description should be updated to list only confirmed CVEs (or at minimum document the source for each claimed fix).
All checklist items in the PR template are unchecked. For a security-patch backport, please confirm:
- ✅ Branch target is
camel-4.22.x(correct here — this is a backport, the note in the template refers to new feature PRs targetingmain) - The build was run locally (
mvn clean install -DskipTests) — standard requirement for Camel contributions
This review was generated by an AI agent, Hermès on behalf of @gnodet.
davsclaus
left a comment
There was a problem hiding this comment.
TL;DR: LGTM, a CVE-driven patch bump on the LTS branch, to the same versions main has used since late October.
The GitHub advisories (e.g. CVE-2026-91777 / GHSA-cxp5-3px4-pw24, high) affect jackson-databind 2.22.0-2.22.2 and 3.2.0-3.2.2, so this is a security fix as the LTS policy expects. Both changes are patch bumps within 2.22.x / 3.2.x, and main has had 2.22.4 / 3.2.4 since #28290 / #28292. Both BOMs keep jackson-annotations at 2.22, and the version properties are only used as ${...}, so nothing else needs to change. The stale camel-sbom.json is regenerated separately. Thanks!
Claude Code on behalf of davsclaus. This review was generated by an AI agent and may contain inaccuracies; please verify all suggestions before applying.
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
|
🧪 CI tested the following changed modules:
✅ POM dependency changes: targeted tests included Changed properties: jackson2-version,jackson3-version Modules affected by dependency changes (10)
🔬 Scalpel shadow comparison — Scalpel: 40 tested, 29 compile-only — current: 0 all testedMaveniverse Scalpel detected 69 affected modules (current approach: 0).
|
Description
Fixes CVE-2026-68495, CVE-2026-68496, CVE-2026-68497, CVE-2026-68498, CVE-2026-89407, CVE-2026-89425, CVE-2026-91777, CVE-2026-104016, CVE-2026-104895, CVE-2026-106554, CVE-2026-110747, CVE-2026-110879
Target
mainbranch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTestslocally from root folder and I have committed all auto-generated changes.AI-assisted contributions
Co-authored-bytrailers) and the PR description identifies the AI tool used.