fix: make telemetry source a string not an enum (#7390)

kylecarbs · web-flow · commit 90c57a538c5e · 2023-05-03T14:33:51.000Z
diff --git a/coderd/telemetry/telemetry.go b/coderd/telemetry/telemetry.go
@@ -233,8 +233,8 @@ func (r *remoteReporter) deployment() error {
 
 	// Tracks where Coder was installed from!
 	installSource := os.Getenv("CODER_TELEMETRY_INSTALL_SOURCE")
-	if installSource != "" && installSource != "aws_marketplace" && installSource != "fly.io" {
-		return xerrors.Errorf("invalid installce source: %s", installSource)
+	if len(installSource) > 64 {
+		return xerrors.Errorf("install source must be <=64 chars: %s", installSource)
 	}
 
 	data, err := json.Marshal(&Deployment{

Original file line number	Diff line number	Diff line change
`@@ -233,8 +233,8 @@ func (r *remoteReporter) deployment() error {`
`233`	`233`
`234`	`234`	`// Tracks where Coder was installed from!`
`235`	`235`	`installSource := os.Getenv("CODER_TELEMETRY_INSTALL_SOURCE")`
`236`		`- if installSource != "" && installSource != "aws_marketplace" && installSource != "fly.io" {`
`237`		`- return xerrors.Errorf("invalid installce source: %s", installSource)`
	`236`	`+ if len(installSource) > 64 {`
	`237`	`+ return xerrors.Errorf("install source must be <=64 chars: %s", installSource)`
`238`	`238`	`}`
`239`	`239`
`240`	`240`	`data, err := json.Marshal(&Deployment{`