From 783c66ea0b9fcf374ec57bf34be60a3b01617a22 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Arpad=20M=C3=BCller?= Date: Tue, 15 Jul 2025 19:57:21 +0200 Subject: [PATCH] Log all GET requests in safekeepers, pageservers, etc --- libs/http-utils/src/endpoint.rs | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/libs/http-utils/src/endpoint.rs b/libs/http-utils/src/endpoint.rs index a61bf8e08a..90eb960899 100644 --- a/libs/http-utils/src/endpoint.rs +++ b/libs/http-utils/src/endpoint.rs @@ -81,14 +81,10 @@ where let path = request.uri().path(); let request_span = info_span!("request", %method, %path, %request_id); - let log_quietly = method == Method::GET; async move { let cancellation_guard = RequestCancelled::warn_when_dropped_without_responding(); - if log_quietly { - debug!("Handling request"); - } else { - info!("Handling request"); - } + + info!("Handling request"); // No special handling for panics here. There's a `tracing_panic_hook` from another // module to do that globally. @@ -109,11 +105,7 @@ where match res { Ok(response) => { let response_status = response.status(); - if log_quietly && response_status.is_success() { - debug!("Request handled, status: {response_status}"); - } else { - info!("Request handled, status: {response_status}"); - } + info!("Request handled, status: {response_status}"); Ok(response) } Err(err) => Ok(api_error_handler(err)),