make isCoderModule into a top level function

hugodutka · hugodutka · commit 7b9d70aa3e37 · 2024-11-16T12:19:04.000Z
diff --git a/coderd/telemetry/telemetry.go b/coderd/telemetry/telemetry.go
@@ -676,11 +676,14 @@ func ConvertWorkspaceResourceMetadata(metadata database.WorkspaceResourceMetadat
 	}
 }
 
+func shouldSendRawModuleSource(source string) bool {
+	return strings.Contains(source, "registry.coder.com")
+}
+
 func ConvertWorkspaceModule(module database.WorkspaceModule) WorkspaceModule {
-	isCoderModule := strings.Contains(module.Source, "registry.coder.com")
 	source := module.Source
 	version := module.Version
-	if !isCoderModule {
+	if !shouldSendRawModuleSource(source) {
 		source = fmt.Sprintf("%x", sha256.Sum256([]byte(source)))
 		version = fmt.Sprintf("%x", sha256.Sum256([]byte(version)))
 	}

Original file line number	Diff line number	Diff line change
`@@ -676,11 +676,14 @@ func ConvertWorkspaceResourceMetadata(metadata database.WorkspaceResourceMetadat`
`676`	`676`	`}`
`677`	`677`	`}`
`678`	`678`
	`679`	`+func shouldSendRawModuleSource(source string) bool {`
	`680`	`+ return strings.Contains(source, "registry.coder.com")`
	`681`	`+}`
	`682`	`+`
`679`	`683`	`func ConvertWorkspaceModule(module database.WorkspaceModule) WorkspaceModule {`
`680`		`- isCoderModule := strings.Contains(module.Source, "registry.coder.com")`
`681`	`684`	`source := module.Source`
`682`	`685`	`version := module.Version`
`683`		`- if !isCoderModule {`
	`686`	`+ if !shouldSendRawModuleSource(source) {`
`684`	`687`	`source = fmt.Sprintf("%x", sha256.Sum256([]byte(source)))`
`685`	`688`	`version = fmt.Sprintf("%x", sha256.Sum256([]byte(version)))`
`686`	`689`	`}`