fix: incomplete regular expression for hostnames (#263)

iainlane · github-advanced-security[bot] · web-flow · commit 29ba642a66d7 · 2025-05-28T09:48:52.000+01:00
CodeQL detected where we'd missed escaping a `.`, which means we match any character instead of a literal `.`, as was the intention. Copilot Autofix fixed it! Closes: #170 Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
diff --git a/pkg/handlers/launch.go b/pkg/handlers/launch.go
@@ -28,7 +28,7 @@ const (
 )
 
 // https://regex101.com/r/OZwd8Y/1
-var outputRegex = regexp.MustCompile(`output: cloud \((?P<url>https:\/\/((app\.k6\.io)|([^/]+\.grafana.net\/a\/k6-app))\/runs\/\d+)\)`)
+var outputRegex = regexp.MustCompile(`output: cloud \((?P<url>https:\/\/((app\.k6\.io)|([^/]+\.grafana\.net\/a\/k6-app))\/runs\/\d+)\)`)
 
 type launchPayload struct {
 	flaggerWebhook

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ const (`
`28`	`28`	`)`
`29`	`29`
`30`	`30`	`// https://regex101.com/r/OZwd8Y/1`
`31`		-var outputRegex = regexp.MustCompile(`output: cloud \((?P<url>https:\/\/((app\.k6\.io)\|([^/]+\.grafana.net\/a\/k6-app))\/runs\/\d+)\)`)
	`31`	+var outputRegex = regexp.MustCompile(`output: cloud \((?P<url>https:\/\/((app\.k6\.io)\|([^/]+\.grafana\.net\/a\/k6-app))\/runs\/\d+)\)`)
`32`	`32`
`33`	`33`	`type launchPayload struct {`
`34`	`34`	`flaggerWebhook`