Skip the transformer if log4j-api isn't present

Signed-off-by: Ralph Goers <[email protected]>
apache · Oct 15, 2024 · fbf2ff7 · fbf2ff7
1 parent ff0fd35
commit fbf2ff7
Showing 1 changed file with 15 additions and 5 deletions.
diff --git a/...orm-maven-plugin/src/main/java/org/apache/logging/log4j/transform/maven/LocationMojo.java b/...orm-maven-plugin/src/main/java/org/apache/logging/log4j/transform/maven/LocationMojo.java
@@ -29,6 +29,7 @@
 import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
+import java.util.Optional;
 import java.util.Set;
 import java.util.stream.Collectors;
 import org.apache.logging.log4j.transform.maven.scan.ClassFileInclusionScanner;
@@ -106,7 +107,9 @@ public void execute() throws MojoExecutionException, MojoFailureException {
             getLog().info("Skipping project with packaging \"pom\".");
             return;
         }
-        validateLog4jVersion();
+        if (!validateLog4jVersion()) {
+            return;
+        }
 
         final Path sourceDirectory = this.sourceDirectory.toPath();
         final Path outputDirectory = this.outputDirectory.toPath();
@@ -189,11 +192,17 @@ private WrappedIOException(IOException cause) {
         }
     }
 
-    private void validateLog4jVersion() throws MojoExecutionException {
-        Artifact log4jApi = project.getArtifacts().stream()
+    private boolean validateLog4jVersion() throws MojoExecutionException {
+        Optional<Artifact> artifact = project.getArtifacts().stream()
                 .filter(a -> LOG4J_GROUP_ID.equals(a.getGroupId()) && LOG4J_API_ARTIFACT_ID.equals(a.getArtifactId()))
-                .findAny()
-                .orElseThrow(() -> new MojoExecutionException("Missing `log4j-api` dependency."));
+                .findAny();
+        Artifact log4jApi;
+        if (artifact.isPresent()) {
+            log4jApi = artifact.get();
+        } else {
+            getLog().info("Skipping project. Log4j is not being used.");
+            return false;
+        }
         try {
             if (MIN_SUPPORTED_VERSION.compareTo(log4jApi.getSelectedVersion()) > 0) {
                 throw new MojoExecutionException("Log4j2 API version " + MIN_SUPPORTED_VERSION
@@ -206,6 +215,7 @@ private void validateLog4jVersion() throws MojoExecutionException {
         } catch (OverConstrainedVersionException e) {
             throw new MojoExecutionException("Can not determine `log4j-api` version.", e);
         }
+        return true;
     }
 
     private ClassLoader getProjectDependencies() throws MojoExecutionException {