Skip to content

build: upgrade docx4j to 11.5.14 - #207

Open
javier-godoy wants to merge 2 commits into
masterfrom
issue-206
Open

build: upgrade docx4j to 11.5.14#207
javier-godoy wants to merge 2 commits into
masterfrom
issue-206

Conversation

@javier-godoy

@javier-godoy javier-godoy commented Sep 9, 2026

Copy link
Copy Markdown
Member

Close #206

Address CVE-2026-53752, a stack overflow in docx4j-core when loading a docx with a cyclic w:basedOn style chain. The vulnerability is not reachable through this add-on, which only generates documents, but downstream applications may interact with docx4j differently.

Summary by CodeRabbit

  • Chores
    • Updated the document-processing components to version 11.5.14.
    • Aligned related document export and JAXB integrations to the same version.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 7f6bd9a6-505a-4fd3-a434-9b4ca75481bb

📥 Commits

Reviewing files that changed from the base of the PR and between 3bd8a16 and 680e51e.

📒 Files selected for processing (1)
  • pom.xml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The Maven configuration introduces docx4j version 11.5.14 and applies it to the JAXB ReferenceImpl, JAXB MOXy, and export-FO dependencies.

Changes

Docx4j dependency upgrade

Layer / File(s) Summary
Version property and dependency references
pom.xml
The POM defines docx4j.version as 11.5.14 and uses it for the three docx4j dependencies.

Priority: ⬆️ High

Estimated code review effort: 1 (Trivial) | ~5 minutes

Severity of issue fixed: High

Merge Risk: ⚪ Minimal · up to 680e5

This change upgrades and aligns docx4j dependencies to version 11.5.14, addressing the targeted vulnerable version without introducing an identified compatibility or runtime risk.

Suggested reviewers: flowingcodeci

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The pull request upgrades all listed docx4j dependencies from 11.5.0 to 11.5.14, including the transitive dependency path through docx4j-JAXB-ReferenceImpl. This addresses issue #206 by upgrading docx…
Out of Scope Changes check ✅ Passed The changes are limited to centralizing the docx4j version and updating three docx4j dependencies. All changes support the remediation described in issue #206.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: upgrading docx4j to version 11.5.14.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue-206

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

@javier-godoy
javier-godoy marked this pull request as ready for review September 9, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Do

Development

Successfully merging this pull request may close these issues.

CVE-2026-53752 org.docx4j:docx4j-core

1 participant