Skip to content

build: upgrade docx4j to 11.5.14 - #207

Merged
paodb merged 2 commits into
masterfrom
issue-206
Sep 21, 2026
Merged

paodb merged 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
@javier-godoy
javier-godoy requested a review from paodb September 21, 2026 19:41
@paodb
paodb merged commit e6afaf3 into master Sep 21, 2026
4 checks passed
@paodb
paodb deleted the issue-206 branch September 21, 2026 20:53
@github-project-automation github-project-automation Bot moved this from To Do to Pending release in Flowing Code Addons Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending release

Development

Successfully merging this pull request may close these issues.

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

2 participants