Extract the telemetry details into build-time variables

Signed-off-by: Remy Suen <remy.suen@docker.com>

Author: rcjsuen

.github/workflows/build.yml

@@ -265,7 +265,7 @@ jobs:
 
       - name: Build for ${{ matrix.os }}-${{ matrix.arch }}
         run: |
-          GOOS=${{ matrix.os }} GOARCH=${{ matrix.arch }} BUGSNAG_API_KEY=${{ secrets.BUGSNAG_API_KEY }} make build
+          GOOS=${{ matrix.os }} GOARCH=${{ matrix.arch }} BUGSNAG_API_KEY=${{ secrets.BUGSNAG_API_KEY }} TELEMETRY_ENDPOINT=${{ env.TELEMETRY_ENDPOINT }} TELEMETRY_KEY=${{ secrets.TELEMETRY_KEY }} make build
 
       - name: Rename the binary to include the version number
         run: |

build.sh

@@ -16,6 +16,6 @@ fi
 echo "Building for ${GOOS}-${GOARCH}: ${OUTPUT}"
 
 CGO_ENABLED=0 go build \
-    -ldflags="-X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.Version=$VERSION' -X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.BugSnagAPIKey=$BUGSNAG_API_KEY'" \
+    -ldflags="-X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.Version=$VERSION' -X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.BugSnagAPIKey=$BUGSNAG_API_KEY' -X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.TelemetryEndpoint=$TELEMETRY_ENDPOINT' -X 'github.com/docker/docker-language-server/internal/pkg/cli/metadata.TelemetryKey=$TELEMETRY_KEY'" \
     -o $OUTPUT \
     ./cmd/docker-language-server

internal/pkg/cli/metadata/metadata.go

@@ -5,3 +5,9 @@ var Version = "0.0.0"
 
 // BugSnagAPIKey will be set dynamically by the build.sh script.
 var BugSnagAPIKey = ""
+
+// TelemetryEndpoint will be set dynamically by the build.sh script.
+var TelemetryEndpoint = ""
+
+// TelemetryKey will be set dynamically by the build.sh script.
+var TelemetryKey = ""

internal/telemetry/client.go

@@ -14,8 +14,6 @@ import (
 	"github.com/docker/docker-language-server/internal/pkg/cli/metadata"
 )
 
-const apiKey = "eIxc3dSmud2vuJRKiq9hJ6wORVWfoLxp1nqb4qXz"
-
 type TelemetryClient interface {
 	Enqueue(event string, properties map[string]any)
 	Publish(ctx context.Context) (int, error)
@@ -28,8 +26,6 @@ type TelemetryClientImpl struct {
 	records   []Record
 }
 
-const telemetryUrl = "https://api.docker.com/events/v1/track"
-
 func NewClient() TelemetryClient {
 	return &TelemetryClientImpl{telemetry: configuration.TelemetrySettingAll}
 }
@@ -85,7 +81,7 @@ func (c *TelemetryClientImpl) trimRecords() []Record {
 }
 
 func (c *TelemetryClientImpl) Publish(ctx context.Context) (int, error) {
-	if os.Getenv("DOCKER_LANGUAGE_SERVER_TELEMETRY") == "false" {
+	if os.Getenv("DOCKER_LANGUAGE_SERVER_TELEMETRY") == "false" || metadata.TelemetryEndpoint == "" || metadata.TelemetryKey == "" {
 		c.records = nil
 		return 0, nil
 	}
@@ -102,14 +98,14 @@ func (c *TelemetryClientImpl) Publish(ctx context.Context) (int, error) {
 		return 0, fmt.Errorf("failed to marshal telemetry payload: %w", err)
 	}
 
-	req, err := http.NewRequestWithContext(ctx, http.MethodPost, telemetryUrl, bytes.NewBuffer(b))
+	req, err := http.NewRequestWithContext(ctx, http.MethodPost, metadata.TelemetryEndpoint, bytes.NewBuffer(b))
 	if err != nil {
 		return 0, fmt.Errorf("failed to create telemetry request: %w", err)
 	}
 
 	req.Header.Set("Content-Type", "application/json")
 	req.Header.Set("User-Agent", fmt.Sprintf("docker-language-server/v%v", metadata.Version))
-	req.Header.Set("x-api-key", apiKey)
+	req.Header.Set("x-api-key", metadata.TelemetryKey)
 	res, err := http.DefaultClient.Do(req)
 	if err != nil {
 		return 0, fmt.Errorf("failed to send telemetry request: %w", err)