fix(analytics): check do not track (#4329) (#4330)

Our do not track check was wrong and always returning true. This is why we were getting nothing. (cherry picked from commit e04d0f6) Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Danilo Pantani <[email protected]>
ignite · Aug 29, 2024 · 7e5253c · 7e5253c
1 parent cb7b4a0
commit 7e5253c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/ignite/internal/analytics/analytics.go b/ignite/internal/analytics/analytics.go
@@ -102,8 +102,10 @@ func SendMetric(wg *sync.WaitGroup, cmd *cobra.Command) {
 // checkDNT check if the user allow to track data or if the DO_NOT_TRACK
 // env var is set https://consoledonottrack.com/
 func checkDNT() (anonIdentity, error) {
-	if dnt, err := strconv.ParseBool(os.Getenv(envDoNotTrack)); err != nil || dnt {
-		return anonIdentity{DoNotTrack: true}, nil
+	if dnt := os.Getenv(envDoNotTrack); dnt != "" {
+		if dnt, err := strconv.ParseBool(dnt); err != nil || dnt {
+			return anonIdentity{DoNotTrack: true}, nil
+		}
 	}
 
 	globalPath, err := config.DirPath()