Issue 49447: HTMLDataLoader isHeaderMatch() to catch DOMException for…

… JSoupUtil.convertHtmlToDocument (#5148)
LabKey · Jan 22, 2024 · 7ac77bf · 7ac77bf
1 parent 02b5efe
commit 7ac77bf
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/api/src/org/labkey/api/reader/HTMLDataLoader.java b/api/src/org/labkey/api/reader/HTMLDataLoader.java
@@ -21,6 +21,7 @@
 import org.labkey.api.util.FileType;
 import org.labkey.api.util.StringUtilsLabKey;
 import org.labkey.api.util.JSoupUtil;
+import org.w3c.dom.DOMException;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.w3c.dom.NodeList;
@@ -82,12 +83,19 @@ public boolean isHeaderMatch(@NotNull byte[] header)
             String s = new String(header, StringUtilsLabKey.DEFAULT_CHARSET);
 
             List<String> errors = new ArrayList<>();
-            Document doc = JSoupUtil.convertHtmlToDocument(s, true, errors);
-            if (!errors.isEmpty() || doc == null)
-                return false;
+            try
+            {
+                Document doc = JSoupUtil.convertHtmlToDocument(s, true, errors);
+                if (!errors.isEmpty() || doc == null)
+                    return false;
 
-            // Look for a html>body>table element
-            return findTable(doc) != null;
+                // Look for a html>body>table element
+                return findTable(doc) != null;
+            }
+            catch (DOMException e)
+            {
+                return false;
+            }
         }
     };