From 7cfefdcf8fcb17accf8fcd75d2874a6dc954f30b Mon Sep 17 00:00:00 2001 From: munzzyy Date: Sun, 5 Jul 2026 11:35:37 -0500 Subject: [PATCH] fix(checkincache): remove stray debug fmt.Println in GetFeatureFlag This debug print was left in by #1707 and runs on every feature-flag read, printing the raw checkInFlagsWrapper struct to stdout. It shows up in miniooni/ooniprobe output as an unlabeled line like: {2026-06-25 14:20:32 +0000 UTC map[vanilla_tor_enabled:true ...]} which is confusing on its own and was reported in ooni/probe-cli#1785 as making it look like the wrong experiment had run. --- internal/checkincache/checkincache.go | 1 - 1 file changed, 1 deletion(-) diff --git a/internal/checkincache/checkincache.go b/internal/checkincache/checkincache.go index 2edf1fcee..dd104cb2e 100644 --- a/internal/checkincache/checkincache.go +++ b/internal/checkincache/checkincache.go @@ -51,7 +51,6 @@ func GetFeatureFlag(kvStore model.KeyValueStore, name string, defaultFlag bool) if err := json.Unmarshal(data, &wrapper); err != nil { return defaultFlag // as documented } - fmt.Println(wrapper) if time.Now().After(wrapper.Expire) { return defaultFlag // as documented }