Skip to content

Have org.apache.logging.log4j.util.Base64Util invoke java.util.Base64…#3690

Merged
ppkarwasz merged 1 commit into
apache:2.xfrom
jluehe:base64-signed
May 25, 2025
Merged

Have org.apache.logging.log4j.util.Base64Util invoke java.util.Base64…#3690
ppkarwasz merged 1 commit into
apache:2.xfrom
jluehe:base64-signed

Conversation

@jluehe

@jluehe jluehe commented May 25, 2025

Copy link
Copy Markdown
Contributor

… directly instead of reflectively (#3686)

This pull request fixes issue #3686 by having org.apache.logging.log4j.util.Base64Util invoke java.util.Base64 directly instead of reflectively. This avoids the need to use javax.xml.bind.DataTypeConverter as a fallback, which has been problematic because it is not Jakarta EE9 compliant.

Checklist

  • Base your changes on 2.x branch if you are targeting Log4j 2; use main otherwise
  • ./mvnw verify succeeds (if it fails due to code formatting issues reported by Spotless, simply run ./mvnw spotless:apply and retry)
  • Non-trivial changes contain an entry file in the src/changelog/.2.x.x directory
  • Tests for the changes are provided
  • Commits are signed (optional, but highly recommended)

@github-actions

Copy link
Copy Markdown
Contributor
Job Requested goals Build Tool Version Build Outcome Build Scan®
build-macos-latest clean install 3.9.8 Build Scan PUBLISHED
build-ubuntu-latest clean install 3.9.8 Build Scan PUBLISHED
build-windows-latest clean install 3.9.8 Build Scan PUBLISHED
Generated by gradle/develocity-actions

@ppkarwasz ppkarwasz merged commit fbb3345 into apache:2.x May 25, 2025
@github-project-automation github-project-automation Bot moved this from To triage to Done in Log4j bug tracker May 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants