Merge pull request #1051 from sysadmind/rm-unsanitized-url

SuperQ · web-flow · commit 078972dba4cf · 2025-10-24T08:14:40.000+02:00
Do not log input URL string
diff --git a/collector/util.go b/collector/util.go
@@ -29,7 +29,7 @@ func getURL(ctx context.Context, hc *http.Client, log *slog.Logger, u string) ([
 
 	resp, err := hc.Do(req)
 	if err != nil {
-		return nil, fmt.Errorf("failed to get %s: %v", u, err)
+		return nil, err
 	}
 
 	defer func() {

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ func getURL(ctx context.Context, hc http.Client, log slog.Logger, u string) ([`
`29`	`29`
`30`	`30`	`resp, err := hc.Do(req)`
`31`	`31`	`if err != nil {`
`32`		`- return nil, fmt.Errorf("failed to get %s: %v", u, err)`
	`32`	`+ return nil, err`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`defer func() {`