From bf48c7b1b44a54e1c44587287b185a3f634d15b9 Mon Sep 17 00:00:00 2001
From: Jason Volk <jason@zemos.net>
Date: Tue, 5 Aug 2025 23:33:31 +0000
Subject: [PATCH] Move active and finished request counters out of span field.

Signed-off-by: Jason Volk <jason@zemos.net>
---
 src/router/request.rs | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/src/router/request.rs b/src/router/request.rs
index 224c4eb8..9f33c1dc 100644
--- a/src/router/request.rs
+++ b/src/router/request.rs
@@ -71,19 +71,8 @@ pub(crate) async fn handle(
 	level = "debug",
 	parent = parent,
 	skip_all,
-	fields(
-		active = %services
-			.server
-			.metrics
-			.requests_handle_active
-			.fetch_add(1, Ordering::Relaxed),
-		handled = %services
-			.server
-			.metrics
-			.requests_handle_finished
-			.load(Ordering::Relaxed),
-	)
 )]
+#[allow(unused_variables)]
 async fn execute(
 	// we made a safety contract that Services will not go out of scope
 	// during the request; this ensures a reference is accounted for at
@@ -93,6 +82,13 @@ async fn execute(
 	next: axum::middleware::Next,
 	parent: &Span,
 ) -> Response {
+	#[cfg(debug_assertions)]
+	services
+		.server
+		.metrics
+		.requests_handle_active
+		.fetch_add(1, Ordering::Relaxed);
+
 	#[cfg(debug_assertions)]
 	tuwunel_core::defer! {{
 		_ = services.server