From 57437e4db544cd3d30f6e3349a8c54f599bc993c Mon Sep 17 00:00:00 2001 From: Benjamin Bouvier Date: Wed, 16 Oct 2024 12:37:28 +0200 Subject: [PATCH] chore(timeline): fix instrumentation of `update_event_send_state` This would not report the `txn_id` field because of the `skip_all`. It's actually interesting to also get the error, so I'm only skipping self from now on. --- crates/matrix-sdk-ui/src/timeline/controller/mod.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crates/matrix-sdk-ui/src/timeline/controller/mod.rs b/crates/matrix-sdk-ui/src/timeline/controller/mod.rs index 94ff0944459..930facfa69b 100644 --- a/crates/matrix-sdk-ui/src/timeline/controller/mod.rs +++ b/crates/matrix-sdk-ui/src/timeline/controller/mod.rs @@ -735,7 +735,7 @@ impl TimelineController

{ /// /// If the corresponding local timeline item is missing, a warning is /// raised. - #[instrument(skip_all, fields(txn_id))] + #[instrument(skip(self))] pub(super) async fn update_event_send_state( &self, txn_id: &TransactionId,