Added optimized filter path for HTTPPathMetricFilter cases (like operator) without need for expensive path parameter resolution

Author: Samin Rahman

src/main/java/com/uid2/shared/util/HTTPPathMetricFilter.java

@@ -6,7 +6,13 @@
 public class HTTPPathMetricFilter {
     public static String filterPath(String actualPath, Set<String> pathSet) {
         try {
-            String normalized = HttpUtils.normalizePath(actualPath).split("\\?")[0];
+            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);
             }
@@ -25,4 +31,29 @@ public static String filterPath(String actualPath, Set<String> pathSet) {
             return "/parsing_error";
         }
     }
+
+    public static String filterPathWithoutPathParameters(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/shared/util/HTTPPathMetricFilterTest.java

@@ -8,7 +8,8 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class HTTPPathMetricFilterTest {
+public class HTTPPathMetricFilterTest {    
+    /* filterPathTests */
     final Set<String> pathSet = Set.of("/v1/identity/map", "/token/refresh", "/list", "/list/:siteId/:keyId");
 
     @ParameterizedTest
@@ -54,4 +55,49 @@ void testPathFiltering_ValidPaths_KnownEndpoints(String actualPath, String expec
         String filteredPath = HTTPPathMetricFilter.filterPath(actualPath, pathSet);
         assertEquals(expectedFilteredPath, filteredPath);
     }
+
+    /* filterPathWithoutPathParameters tests */
+    final Set<String> pathSetWithoutParams = Set.of("/v1/identity/map", "/token/refresh", "/list");
+
+    @ParameterizedTest
+    @ValueSource(strings = {
+            "",
+            "/",
+            "/unknown-path",
+            "../",
+            "/v1/identity/map%55",
+            "/list/123",
+    })
+    void testPathFilteringWithoutPathParameters_InvalidPaths_Unknown(String actualPath) {
+        String filteredPath = HTTPPathMetricFilter.filterPathWithoutPathParameters(actualPath, pathSetWithoutParams);
+        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 testPathFilteringWithoutPathParameters_InvalidPaths_ParsingError(String actualPath) {
+        String filteredPath = HTTPPathMetricFilter.filterPathWithoutPathParameters(actualPath, pathSetWithoutParams);
+        assertEquals("/parsing_error", filteredPath);
+    }
+
+    @ParameterizedTest
+    @CsvSource(value = {
+            "/v1/identity/map, /v1/identity/map",
+            "v1/identity/map, /v1/identity/map",
+            "V1/IdenTity/mAp, /v1/identity/map",
+            "./v1//identity//map/, /v1/identity/map",
+            "../v1/identity/./map, /v1/identity/map",
+            "/v1/identity/new/path/../../map, /v1/identity/map",
+            "token/refresh?refresh_token=123%20%23, /token/refresh",
+            "v1/identity/map?identity/../map/, /v1/identity/map",
+            "/list, /list"
+    })
+    void testPathFilteringWithoutPathParameters_ValidPaths_KnownEndpoints(String actualPath, String expectedFilteredPath) {
+        String filteredPath = HTTPPathMetricFilter.filterPathWithoutPathParameters(actualPath, pathSetWithoutParams);
+        assertEquals(expectedFilteredPath, filteredPath);
+    }
 }