Created a more optimized version of HTTPPathMetricFilter

Author: RSam25

src/main/java/com/uid2/operator/Main.java

@@ -12,6 +12,7 @@
 import com.uid2.operator.service.*;
 import com.uid2.operator.store.*;
 import com.uid2.operator.store.BootstrapConfigStore;
+import com.uid2.operator.util.HTTPPathMetricFilterOptimized;
 import com.uid2.operator.vertx.Endpoints;
 import com.uid2.operator.vertx.OperatorShutdownHandler;
 import com.uid2.operator.vertx.UIDOperatorVerticle;
@@ -517,7 +518,7 @@ private static void setupMetrics(MicrometerMetricsOptions metricOptions) {
                 // providing common renaming for prometheus metric, e.g. "hello.world" to "hello_world"
                 .meterFilter(new PrometheusRenameFilter())
                 .meterFilter(MeterFilter.replaceTagValues(Label.HTTP_PATH.toString(),
-                        actualPath -> HTTPPathMetricFilter.filterPath(actualPath, Endpoints.pathSet())))
+                        actualPath -> HTTPPathMetricFilterOptimized.filterPath(actualPath, Endpoints.pathSet())))
                 // Don't record metrics for 404s.
                 .meterFilter(MeterFilter.deny(id ->
                     id.getName().startsWith(MetricsDomain.HTTP_SERVER.getPrefix()) &&

src/main/java/com/uid2/operator/util/HTTPPathMetricFilterOptimized.java

@@ -0,0 +1,32 @@
+package com.uid2.operator.util;
+
+import io.vertx.core.http.impl.HttpUtils;
+
+import java.util.Set;
+
+public class HTTPPathMetricFilterOptimized {
+    public static String filterPath(String actualPath, Set<String> pathSet) {
+        try {
+            String normalized = HttpUtils.normalizePath(actualPath);
+            /* Optimization 1: Split that avoids array and regex initialization */
+            int splitIndex = normalized.indexOf('?');
+            if (splitIndex != -1) {
+                normalized = normalized.substring(0, splitIndex);
+            }
+
+            if (normalized.charAt(normalized.length() - 1) == '/') {
+                normalized = normalized.substring(0, normalized.length() - 1);
+            }
+            normalized = normalized.toLowerCase();
+
+            if (pathSet == null || pathSet.isEmpty()) { return normalized; }
+
+            /* Optimization 2: Remove for loop and regex matching */
+            if (pathSet.contains(normalized)) { return normalized; }
+
+            return "/unknown";
+        } catch (IllegalArgumentException e) {
+            return "/parsing_error";
+        }
+    }
+}

src/test/java/com/uid2/operator/util/HTTPPathMetricFilterOptimizedTest.java

@@ -0,0 +1,48 @@
+package com.uid2.operator.util;
+
+import com.uid2.operator.vertx.Endpoints;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.CsvSource;
+import org.junit.jupiter.params.provider.ValueSource;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class HTTPPathMetricFilterOptimizedTest {
+    @ParameterizedTest
+    @ValueSource(strings = {
+            "",
+            "/",
+            "/unknown-path",
+            "../",
+            "/v1/identity/map%55",
+            "/list/123",
+    })
+    void testPathFiltering_InvalidPaths_Unknown(String actualPath) {
+        String filteredPath = HTTPPathMetricFilterOptimized.filterPath(actualPath, Endpoints.pathSet());
+        assertEquals("/unknown", filteredPath);
+    }
+
+    @ParameterizedTest
+    @ValueSource(strings = {
+            "v1/identity/map?id=bad-escape-code%2",
+            "token/refresh?refresh_token=SOME_TOKEN<%=7485*4353%>",
+            "list/12%4/5435"
+    })
+    void testPathFiltering_InvalidPaths_ParsingError(String actualPath) {
+        String filteredPath = HTTPPathMetricFilterOptimized.filterPath(actualPath, Endpoints.pathSet());
+        assertEquals("/parsing_error", filteredPath);
+    }
+
+    @ParameterizedTest
+    @CsvSource(value = {
+            "/v2/identity/map, /v2/identity/map",
+            "v2/identity/map, /v2/identity/map",
+            "V3/IdenTity/mAp, /v3/identity/map",
+            "v2/token/refresh?refresh_token=123%20%23, /v2/token/refresh",
+            "v2/identity/map?identity/../map/, /v2/identity/map"
+    })
+    void testPathFiltering_ValidPaths_KnownEndpoints(String actualPath, String expectedFilteredPath) {
+        String filteredPath = HTTPPathMetricFilterOptimized.filterPath(actualPath, Endpoints.pathSet());
+        assertEquals(expectedFilteredPath, filteredPath);
+    }
+}