diff --git a/pom.xml b/pom.xml index 2b9a4695..f1d9f25c 100644 --- a/pom.xml +++ b/pom.xml @@ -16,7 +16,7 @@ 4.5.11 1.1.0 - 8.0.6 + 8.0.32 ${project.version} 5.10.1 5.10.1 diff --git a/src/main/java/com/uid2/optout/Main.java b/src/main/java/com/uid2/optout/Main.java index df9b846e..160882ea 100644 --- a/src/main/java/com/uid2/optout/Main.java +++ b/src/main/java/com/uid2/optout/Main.java @@ -17,6 +17,7 @@ import com.uid2.shared.vertx.CloudSyncVerticle; import com.uid2.shared.vertx.RotatingStoreVerticle; import com.uid2.shared.vertx.VertxUtils; +import com.uid2.shared.util.HTTPPathMetricFilter; import io.micrometer.core.instrument.Gauge; import io.micrometer.core.instrument.Metrics; import io.micrometer.core.instrument.config.MeterFilter; @@ -218,14 +219,8 @@ private static void setupMetrics(MicrometerMetricsOptions metricOptions) { prometheusRegistry.config() // providing common renaming for prometheus metric, e.g. "hello.world" to "hello_world" .meterFilter(new PrometheusRenameFilter()) - .meterFilter(MeterFilter.replaceTagValues(Label.HTTP_PATH.toString(), actualPath -> { - try { - String normalized = HttpUtils.normalizePath(actualPath).split("\\?")[0]; - return Endpoints.pathSet().contains(normalized) ? normalized : "/unknown"; - } catch (IllegalArgumentException e) { - return actualPath; - } - })) + .meterFilter(MeterFilter.replaceTagValues(Label.HTTP_PATH.toString(), + actualPath -> HTTPPathMetricFilter.filterPath(actualPath, Endpoints.pathSet()))) // Don't record metrics for 404s. .meterFilter(MeterFilter.deny(id -> id.getName().startsWith(MetricsDomain.HTTP_SERVER.getPrefix()) &&