Merge pull request #33595 from aloubyansky/manifest-entry-npe

Skip manifest entry properties that are missing values
quarkusio · May 26, 2023 · a76d1b5 · a76d1b5
2 parents ff60ec9 + 0e9a248
commit a76d1b5
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/devtools/maven/src/main/java/io/quarkus/maven/QuarkusBootstrapProvider.java b/devtools/maven/src/main/java/io/quarkus/maven/QuarkusBootstrapProvider.java
@@ -197,8 +197,11 @@ private CuratedApplication doBootstrap(QuarkusBootstrapMojo mojo, LaunchMode mod
             effectiveProperties.putIfAbsent("quarkus.application.version", mojo.mavenProject().getVersion());
 
             for (Map.Entry<String, String> attribute : mojo.manifestEntries().entrySet()) {
-                effectiveProperties.put(toManifestAttributeKey(attribute.getKey()),
-                        attribute.getValue());
+                if (attribute.getValue() == null) {
+                    mojo.getLog().warn("Skipping manifest entry property " + attribute.getKey() + " with a missing value");
+                } else {
+                    effectiveProperties.put(toManifestAttributeKey(attribute.getKey()), attribute.getValue());
+                }
             }
             for (ManifestSection section : mojo.manifestSections()) {
                 for (Map.Entry<String, String> attribute : section.getManifestEntries().entrySet()) {