From eba88ed7bb9999ce83c6bbf5e8b134b9fbc0beeb Mon Sep 17 00:00:00 2001 From: Louis Knight-Webb Date: Mon, 16 Mar 2026 16:55:43 +0000 Subject: [PATCH] Disable remote analytics when PostHog config is blank (Vibe Kanban) (#3161) * Disable remote analytics when PostHog config is blank * Remove analytics config unit tests --- crates/remote/src/analytics.rs | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/crates/remote/src/analytics.rs b/crates/remote/src/analytics.rs index 2cbad86d5a..602599a564 100644 --- a/crates/remote/src/analytics.rs +++ b/crates/remote/src/analytics.rs @@ -11,11 +11,23 @@ pub struct AnalyticsConfig { impl AnalyticsConfig { pub fn from_env() -> Option { - let api_key = option_env!("POSTHOG_API_KEY")?.to_string(); - let api_endpoint = option_env!("POSTHOG_API_ENDPOINT")?.to_string(); + Self::from_values( + option_env!("POSTHOG_API_KEY"), + option_env!("POSTHOG_API_ENDPOINT"), + ) + } + + fn from_values(api_key: Option<&str>, api_endpoint: Option<&str>) -> Option { + let api_key = api_key?.trim(); + let api_endpoint = api_endpoint?.trim(); + + if api_key.is_empty() || api_endpoint.is_empty() { + return None; + } + Some(Self { - posthog_api_key: api_key, - posthog_api_endpoint: api_endpoint, + posthog_api_key: api_key.to_string(), + posthog_api_endpoint: api_endpoint.to_string(), }) } }