From caafcbcf0b637c0ae2faf889195113ff82c4ff10 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Fri, 6 Jun 2025 15:58:16 +0200 Subject: [PATCH] fix(gui-client): apply log directives from MDM config (#9443) When introducing the MDM config, we took into account the log directives from the config when applying it via the GUI but failed to apply the new directives on startup. --- rust/gui-client/src-tauri/src/gui.rs | 6 ++++++ website/src/components/Changelog/GUI.tsx | 7 ++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/rust/gui-client/src-tauri/src/gui.rs b/rust/gui-client/src-tauri/src/gui.rs index 76352f4f6..0f372373d 100644 --- a/rust/gui-client/src-tauri/src/gui.rs +++ b/rust/gui-client/src-tauri/src/gui.rs @@ -235,6 +235,12 @@ pub fn run( .inspect_err(|e| tracing::debug!("Failed to load MDM settings {e:#}")) .unwrap_or_default(); + if let Some(directives) = mdm_settings.log_filter.as_ref() { + if let Err(e) = reloader.reload(directives) { + tracing::info!(%directives, "Failed to apply MDM logging directives: {e:#}"); + } + } + let api_url = mdm_settings .api_url .as_ref() diff --git a/website/src/components/Changelog/GUI.tsx b/website/src/components/Changelog/GUI.tsx index 3940ad27c..389b6ac9b 100644 --- a/website/src/components/Changelog/GUI.tsx +++ b/website/src/components/Changelog/GUI.tsx @@ -10,7 +10,12 @@ export default function GUI({ os }: { os: OS }) { return ( {/* When you cut a release, remove any solved issues from the "known issues" lists over in `client-apps`. This must not be done when the issue's PR merges. */} - + + + Fixes an issue where log directives applied via MDM would not be + applied on startup. + + Fixes an issue where advanced settings were not saved and loaded