actix-web-prom

actix-web-prom copied to clipboard

Rename the default histogram

2

By renaming this metric we are making the package comply with Prometheus naming convention. Reference: https://prometheus.io/docs/practices/naming/

albertogg

feat: configurable cardinality for endpoint pattern label

8

Hello, first thanks for this wonderful crate. Like discussed in #20 and implemented in #30 we currently try to get the match_pattern of the request and use that as the...

lefuturiste

This change breaks the API slightly - users need to access registry by `.registry.as_ref()` instead of just `.registry`

artefom

PrometheusMetricsBuilder consumes Registry not allowing to pass default prometheus registry. This prevents usage of very convinient lazy static metrics from prometheus package ```rust // This will register to prometheus::default_registry() lazy_static!...

artefom

Question: How to avoid `WARN actix_web_prom: Cannot build mixed cardinality pattern` ?

4

Hi, I'm excluding `swagger-ui` but I'm still seeing a warning in logs ```rust let metrics = PrometheusMetricsBuilder::new(namespace.as_ref()) .registry(registry.into()) .exclude("/swagger-ui") .exclude_regex("/swagger-ui/.*") .build(); ``` ```console 2024-05-13T14:04:36.844346Z WARN actix_web_prom: Cannot build mixed cardinality...

helio-frota

Adding MetricsConfiguratorMiddleware to allow upstream code to provide a quick way to configure cardinality_keep_params with wrap

lefuturiste