Enhanced the MX parser log verbosity when parsing malformed content

Author: zubri

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Prowide ISO 20022 - CHANGELOG
 
+#### 9.5.7 - SNAPSHOT
+  * Enhanced the MX parser log verbosity when parsing malformed content
+
 #### 9.5.6 - January 2025
   * Changed the `MxParseUtils` findByTags and findByPath methods to return the element values instead of the XML stream object
 

iso20022-core/src/main/java/com/prowidesoftware/swift/model/mx/MxReadImpl.java

@@ -93,6 +93,10 @@ public static AbstractMX parse(
 
             return mx.orElse(null);
 
+        } catch (AssertionError e) {
+            // depending on the jaxb implementation used, the assertion error can be thrown when the XML is not valid
+            log.log(Level.SEVERE, "Error parsing XML", e);
+            return null;
         } catch (final Exception e) {
             MxParseUtils.handleParseException(e);
             return null;

iso20022-core/src/main/java/com/prowidesoftware/swift/model/mx/NamespaceAndElementFilter.java

@@ -45,14 +45,14 @@
  * @since 9.2.1
  */
 public class NamespaceAndElementFilter extends XMLFilterImpl {
-    private static final transient Logger log = Logger.getLogger(NamespaceAndElementFilter.class.getName());
+    private static final Logger log = Logger.getLogger(NamespaceAndElementFilter.class.getName());
 
     private String mainNamespace;
     private boolean inElementToPropagate = false;
-    private String localNameToPropagate;
+    private final String localNameToPropagate;
     private boolean inInnerElementToSkip = false;
     private String localNameToSkip;
-    private boolean unbindNamespace;
+    private final boolean unbindNamespace;
 
     /**
      * @param localName the XML's element to propagate
@@ -91,7 +91,10 @@ public void startElement(String namespace, String localName, String prefix, Attr
                 try {
                     super.startElement(namespaceToPropagate, localName, prefix, attributes);
                 } catch (Exception e) {
-                    log.log(Level.WARNING, "Error parsing " + localName + " [" + namespace + "] element", e);
+                    log.warning("Error parsing " + localName + " [" + namespace + "] element: " + exceptionMessage(e));
+                    if (log.isLoggable(Level.FINEST)) {
+                        log.log(Level.FINEST, "Error parsing " + localName + " [" + namespace + "] element", e);
+                    }
                 }
             } else {
                 // we have found an element within the structure to propagate with a not recognized namespace
@@ -150,7 +153,10 @@ public void endElement(String namespace, String localName, String prefix) throws
                 try {
                     super.endElement(namespaceToPropagate, localName, prefix);
                 } catch (Exception e) {
-                    log.log(Level.WARNING, "Error parsing " + localName + " [" + namespace + "] element", e);
+                    log.warning("Error parsing " + localName + " [" + namespace + "] element: " + exceptionMessage(e));
+                    if (log.isLoggable(Level.FINEST)) {
+                        log.log(Level.FINEST, "Error parsing " + localName + " [" + namespace + "] element", e);
+                    }
                 }
             }
         }
@@ -169,9 +175,23 @@ public void startPrefixMapping(String prefix, String url) throws SAXException {
                 try {
                     super.startPrefixMapping(prefix, url);
                 } catch (Exception e) {
-                    log.log(Level.WARNING, "Error parsing " + prefix + " [" + url + "] prefix mapping", e);
+                    log.warning("Error parsing " + prefix + " [" + url + "] prefix mapping: " + exceptionMessage(e));
+                    if (log.isLoggable(Level.FINEST)) {
+                        log.log(Level.FINEST, "Error parsing " + prefix + " [" + url + "] prefix mapping", e);
+                    }
                 }
             }
         }
     }
+
+    private static String exceptionMessage(Exception e) {
+        String message = e.getMessage();
+        if (message == null && e.getCause() != null) {
+            message = e.getCause().getMessage();
+        }
+        if (message == null) {
+            message = e.getClass().getSimpleName();
+        }
+        return message;
+    }
 }