Skip to content

Align all log4j artifacts to 2.25.3 (supersedes #5) - #8

Merged
Mangesh Sangapu (msangapu-msft) merged 2 commits into
mainfrom
lu-log4j-align
Jul 23, 2026
Merged

Align all log4j artifacts to 2.25.3 (supersedes #5)#8
Mangesh Sangapu (msangapu-msft) merged 2 commits into
mainfrom
lu-log4j-align

Conversation

@cephalin

Copy link
Copy Markdown
Contributor

Supersedes #5.

Dependabot PR #5 bumped only log4j-core (2.22.1 → 2.25.3) and left log4j-api and log4j-jul at 2.22.1, leaving the three log4j artifacts at mismatched versions.

This PR introduces a shared log4j.version property (2.25.3) and references it from all three dependencies so they always move together.

Changes

  • Add <log4j.version>2.25.3</log4j.version> property
  • log4j-core, log4j-api, log4j-jul all use ${log4j.version}

Validation

  • All three artifacts confirmed published at 2.25.3 on Maven Central (HTTP 200)
  • pom.xml is well-formed XML
  • Source uses only stable public log4j API (LogManager.getLogger, logger.info) and a standard log4j2.properties config — no breaking changes across 2.22 → 2.25

dependabot Bot and others added 2 commits December 19, 2025 23:42
Bumps org.apache.logging.log4j:log4j-core from 2.22.1 to 2.25.3.

---
updated-dependencies:
- dependency-name: org.apache.logging.log4j:log4j-core
  dependency-version: 2.25.3
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Introduces a log4j.version property (2.25.3) and references it from
log4j-core, log4j-api, and log4j-jul so the three artifacts always move
together. This completes Dependabot PR #5, which bumped only log4j-core
and left log4j-api/log4j-jul at 2.22.1.

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

This PR aligns the Log4j 2 dependencies used by the sample app by introducing a shared Maven property so log4j-core, log4j-api, and log4j-jul are always kept on the same version.

Changes:

  • Add a log4j.version Maven property set to 2.25.3.
  • Update log4j-core, log4j-api, and log4j-jul dependencies to reference ${log4j.version}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@msangapu-msft
Mangesh Sangapu (msangapu-msft) merged commit a89b944 into main Jul 23, 2026
2 checks passed
@cephalin
Cephas Lin (cephalin) deleted the lu-log4j-align branch July 23, 2026 15:50
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