Merge pull request #402 from cherylking/anotherEmptyFeatureWarning

Check for empty feature
OpenLiberty · Jun 5, 2023 · 4e720b6 · 4e720b6
2 parents 59bb5d9 + 39416de
commit 4e720b6
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/main/java/io/openliberty/tools/common/plugins/util/ServerFeatureUtil.java b/src/main/java/io/openliberty/tools/common/plugins/util/ServerFeatureUtil.java
@@ -371,7 +371,12 @@ private Set<String> parseFeatureManagerNode(Element node) {
                         } else {
                             content = content.trim();
                         }
-                        result.add(content);
+                        // Check for empty feature element, skip it and log warning.
+                        if (content.isEmpty()) {
+                            warn("An empty feature was specified in a server configuration file. Ensure that the features are valid.");
+                        } else {
+                            result.add(content);
+                        }
                 	}
                 }
             }