Loosen regex for agent attribute (#55)

the-ericwang35 · web-flow · commit 649245c436a0 · 2020-08-07T17:42:42.000Z
diff --git a/internal/validation/mock_trace_validation.go b/internal/validation/mock_trace_validation.go
@@ -54,7 +54,7 @@ var (
 	}
 	requiredFields = []string{"Name", "SpanId", "DisplayName", "StartTime", "EndTime"}
 	spanNameRegex  = regexp.MustCompile("^projects/[^/]+/traces/[a-fA-F0-9]{32}/spans/[a-fA-F0-9]{16}$")
-	agentRegex     = regexp.MustCompile(`^opentelemetry-[a-zA-Z]+ [0-9]+\.[0-9]+\.[0-9]+; google-cloud-trace-exporter [0-9]+\.[0-9]+\.[0-9]+$`)
+	agentRegex     = regexp.MustCompile(`^opentelemetry-[a-zA-Z]+ \d+(?:\.\d+)+; google-cloud-trace-exporter \d+(?:\.\d+)+$`)
 )
 
 // SpanData wraps all the span data on the server into a struct.

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ var (`
`54`	`54`	`}`
`55`	`55`	`requiredFields = []string{"Name", "SpanId", "DisplayName", "StartTime", "EndTime"}`
`56`	`56`	`spanNameRegex = regexp.MustCompile("^projects/[^/]+/traces/[a-fA-F0-9]{32}/spans/[a-fA-F0-9]{16}$")`
`57`		- agentRegex = regexp.MustCompile(`^opentelemetry-[a-zA-Z]+ [0-9]+\.[0-9]+\.[0-9]+; google-cloud-trace-exporter [0-9]+\.[0-9]+\.[0-9]+$`)
	`57`	+ agentRegex = regexp.MustCompile(`^opentelemetry-[a-zA-Z]+ \d+(?:\.\d+)+; google-cloud-trace-exporter \d+(?:\.\d+)+$`)
`58`	`58`	`)`
`59`	`59`
`60`	`60`	`// SpanData wraps all the span data on the server into a struct.`