Merge pull request #5 from rapid7/DCA-22136

DCA-22136 Fix empty name parameter
rapid7 · Jan 15, 2021 · 0fced36 · 0fced36
2 parents 257bf77 + cadc60f
commit 0fced36
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/main/java/com/rapid7/recog/parser/RecogParser.java b/src/main/java/com/rapid7/recog/parser/RecogParser.java
@@ -73,7 +73,7 @@ public RecogMatchers parse(File file)
    *         if no matchers are defined, or all matchers are invalid and strict mode is disabled.
    * @throws ParseException If an error is encountered and strict-mode is enabled.
    */
-  public RecogMatchers parse(Reader reader, String name) // TODO: what was name meant to be used for?
+  public RecogMatchers parse(Reader reader, String name)
       throws ParseException {
     Document document;
     try {
@@ -99,10 +99,9 @@ public RecogMatchers parse(Reader reader, String name) // TODO: what was name me
 
     String recogKey = root.getAttribute("matches");
 
-    if (recogKey.isEmpty() || recogKey == null)
-    {
-       LOGGER.warn("Recog Matcher Key is Empty or Null. File Name: " + name);
-       recogKey = name;
+    if (recogKey.isEmpty() || recogKey == null) {
+      LOGGER.debug("Recog Matcher Key is Empty or Null. File Name: " + name);
+      recogKey = name;
     }
 
     RecogMatchers matchers = new RecogMatchers(recogKey, root.getAttribute("protocol"), root.getAttribute("database_type"), preference);