SonarQube fixes

Author: germanosin

api/src/main/java/io/kafbat/ui/service/metrics/scrape/jmx/JmxMetricsFormatter.java

@@ -18,7 +18,7 @@ public class JmxMetricsFormatter {
 
   // copied from https://github.com/prometheus/jmx_exporter/blob/b6b811b4aae994e812e902b26dd41f29364c0e2b/collector/src/main/java/io/prometheus/jmx/JmxMBeanPropertyCache.java#L15
   private static final Pattern PROPERTY_PATTERN = Pattern.compile(
-      "([^,=:\\*\\?]+)=(\"(?:\\\\.|[^\\\\\"])*\"|[^,=:\"]*)"
+      "([^,=:\\*\\?]+)=(\"(?:[^\\\\\"]*(?:\\\\.)?)*\"|[^,=:\"]*)"
   );
 
   private JmxMetricsFormatter() {
@@ -73,6 +73,11 @@ private static Optional<BigDecimal> convertNumericValue(Object value) {
   private static LinkedHashMap<String, String> getLabelsMap(ObjectName mbeanName) {
     LinkedHashMap<String, String> keyProperties = new LinkedHashMap<>();
     String properties = mbeanName.getKeyPropertyListString();
+
+    if (properties.length() > 1024) {
+      throw new IllegalArgumentException("MBean key property list too long: " + properties);
+    }
+
     Matcher match = PROPERTY_PATTERN.matcher(properties);
     while (match.lookingAt()) {
       String labelName = fixIllegalChars(match.group(1)); // label names should be fixed

api/src/main/java/io/kafbat/ui/service/metrics/scrape/prometheus/PrometheusTextFormatParser.java

@@ -40,9 +40,9 @@ public class PrometheusTextFormatParser {
   // Groups: 1=name, 2=labels (content), 3=value, 4=timestamp
   private static final Pattern METRIC_LINE_PATTERN = Pattern.compile(
       "^([a-zA-Z_:][a-zA-Z0-9_:]*)"            // Metric name
-          + "(?:\\{([^}]*)\\})?"                     // Optional labels (content in group 2)
+          + "(?:\\{([^}]*)})?"                     // Optional labels (content in group 2)
           + "\\s+"
-          + "(-?(?:Inf|NaN|(?:\\d*\\.?\\d+(?:[eE][+-]?\\d+)?)))" // Value (group 3)
+          + "(-?(?:Inf|NaN|\\d*\\.?\\d+(?:[eE][+-]?\\d+)?))" // Value (group 3)
           + "(?:\\s+(\\d+))?$");                 // Group 4: Optional timestamp