awesome-bookmarks/goose
3
0
Fork 0
mirror of https://github.com/block/goose.git synced 2026-07-17 12:56:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

Preserve GOOSE_TELEMETRY_OFF override in set_telemetry_enabled

Browse Source 
The env-var kill switch is a hard disable. Prevent config writes
from re-enabling telemetry when GOOSE_TELEMETRY_OFF is set.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jh-block
2026-03-05 15:23:27 -05:00
parent 71b19f3c09
commit 94eddb247a
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/goose/src/posthog.rs
+3 -1
View File
@@ -59,7 +59,9 @@ pub fn is_telemetry_enabled() -> bool {
/// changed so that Sentry callbacks and other hot paths see the new
/// state immediately.
pub fn set_telemetry_enabled(enabled: bool) {
TELEMETRY_ENABLED.store(enabled, Ordering::Relaxed);
// GOOSE_TELEMETRY_OFF is a hard kill switch; never allow re-enabling.
let effective = enabled && !TELEMETRY_DISABLED_BY_ENV.load(Ordering::Relaxed);
TELEMETRY_ENABLED.store(effective, Ordering::Relaxed);
}
// ============================================================================