From 2f3fe751bfa7416aab682e5aa662798fc81374d4 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Thu, 7 Nov 2024 06:49:23 +1100 Subject: [PATCH] chore(gui-client): log entire error when connlib fails (#7273) The `error_msg` here is already a user-friendly string because we are also showing it to the user in an error message. These can be entirely different errors so we should display them as different messages. This will allow Sentry to group them together correctly. --- rust/gui-client/src-common/src/controller.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust/gui-client/src-common/src/controller.rs b/rust/gui-client/src-common/src/controller.rs index 9f0206f0d..9de0b1e43 100644 --- a/rust/gui-client/src-common/src/controller.rs +++ b/rust/gui-client/src-common/src/controller.rs @@ -539,7 +539,7 @@ impl Controller { "To access resources, sign in again.", )?; } else { - tracing::error!(?error_msg, "Disconnected"); + tracing::error!("Connlib disconnected: {error_msg}"); native_dialog::MessageDialog::new() .set_title("Firezone Error") .set_text(&error_msg)