From f7466d48552e1346b4ac31b45d4ea160a731e31c Mon Sep 17 00:00:00 2001 From: Maidul Islam Date: Fri, 19 May 2023 12:20:37 -0400 Subject: [PATCH] update cli telemetry --- .goreleaser.yaml | 11 ++++++----- cli/packages/telemetry/telemetry.go | 9 ++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/.goreleaser.yaml b/.goreleaser.yaml index 12fb34dba..e30c6b763 100644 --- a/.goreleaser.yaml +++ b/.goreleaser.yaml @@ -15,13 +15,12 @@ monorepo: tag_prefix: infisical-cli/ dir: cli -env: - - POSTHOG_API_KEY_FOR_CLI={{ .Env.POSTHOG_API_KEY_FOR_CLI }} - builds: - id: darwin-build binary: infisical - ldflags: -X github.com/Infisical/infisical-merge/packages/util.CLI_VERSION={{ .Version }} + ldflags: + - -X github.com/Infisical/infisical-merge/packages/util.CLI_VERSION={{ .Version }} + - -X github.com/Infisical/infisical-merge/packages/telemetry.POSTHOG_API_KEY_FOR_CLI={{ .Env.POSTHOG_API_KEY_FOR_CLI }} flags: - -trimpath env: @@ -39,7 +38,9 @@ builds: env: - CGO_ENABLED=0 binary: infisical - ldflags: -X github.com/Infisical/infisical-merge/packages/util.CLI_VERSION={{ .Version }} + ldflags: + - -X github.com/Infisical/infisical-merge/packages/util.CLI_VERSION={{ .Version }} + - -X github.com/Infisical/infisical-merge/packages/telemetry.POSTHOG_API_KEY_FOR_CLI={{ .Env.POSTHOG_API_KEY_FOR_CLI }} flags: - -trimpath goos: diff --git a/cli/packages/telemetry/telemetry.go b/cli/packages/telemetry/telemetry.go index 166b002e1..7b5bd2e5a 100644 --- a/cli/packages/telemetry/telemetry.go +++ b/cli/packages/telemetry/telemetry.go @@ -1,23 +1,22 @@ package telemetry import ( - "os" - "github.com/Infisical/infisical-merge/packages/util" "github.com/denisbrodbeck/machineid" "github.com/posthog/posthog-go" ) +var POSTHOG_API_KEY_FOR_CLI string + type Telemetry struct { isEnabled bool posthogClient posthog.Client } func NewTelemetry(telemetryIsEnabled bool) *Telemetry { - posthogAPIKey := os.Getenv("POSTHOG_API_KEY_FOR_CLI") - if posthogAPIKey != "" { + if POSTHOG_API_KEY_FOR_CLI != "" { client, _ := posthog.NewWithConfig( - posthogAPIKey, + POSTHOG_API_KEY_FOR_CLI, posthog.Config{}, )