[MPIR-448] [REGRESSION] DependenciesRenderer chokes on invalid scope …

…with a NPE
apache · May 31, 2023 · d8f8086 · d8f8086
1 parent 9c25752
commit d8f8086
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/it/full-pom/pom.xml b/src/it/full-pom/pom.xml
@@ -176,6 +176,12 @@
       <version>1.7.36</version>
       <scope>runtime</scope>
     </dependency>
+    <dependency>
+      <groupId>io.netty</groupId>
+      <artifactId>netty-transport-native-kqueue</artifactId>
+      <version>4.1.91.Final</version>
+      <scope>bogus</scope>
+    </dependency>
   </dependencies>
 
   <build>

diff --git a/.../java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java b/.../java/org/apache/maven/report/projectinfo/dependencies/renderer/DependenciesRenderer.java
@@ -533,6 +533,11 @@ private void renderSectionDependencyFileDetails() {
                         + " and won't be listed in dependency files details.");
                 continue;
             }
+            if (SummaryTableRowOrder.fromScope(artifact.getScope()) == null) {
+                log.warn("Artifact " + artifact.getId() + " has invalid scope"
+                        + " and won't be listed in dependency files details.");
+                continue;
+            }
 
             File artifactFile = dependencies.getFile(artifact);