rust-prometheus icon indicating copy to clipboard operation
rust-prometheus copied to clipboard

Registering the same metric multiple times

Open sazzer opened this issue 2 years ago • 1 comments

I've written some Axum middleware to record the number of times each HTTP request is processed, and I'm hitting problems with actually recording the metrics.

What I've done so far is this:

pub async fn record_metrics<B>(
    registry: Extension<prometheus::Registry>,
    request: Request<B>,
    next: Next<B>,
) -> Response {
    let uri = request.uri().to_string();

    let response = next.run(request).await;

    let counter = CounterVec::new(Opts::new("http_request", "HTTP Requests"), &["uri", "status_code"]).unwrap();
    registry.register(Box::new(counter.clone())).unwrap();

    counter.with_label_values(&[&uri, response.status().as_str()]).inc();

    response
}

And the call to registry.register() returns AlreadyReg on all incoming requests after the first one.

Is there some easy way to get back the already registered metric so that I can increment that instead?

I've looked at calling registry.gather() to try and find it, but can't trivially work out how to find the correct entry in the returned set.

Cheers

sazzer avatar Jul 16 '23 10:07 sazzer

Hitting the same situation with #525. We need to be able to get the registered metric from register function. The ideal usage is whenever I call register with legal input, it returns the registered metric, whenever it is just registered, or registered by other call.

sunng87 avatar Jul 06 '24 09:07 sunng87