fix: pointer receiver on a copied struct (#83)

eliottness · web-flow · commit d64577c44af8 · 2024-03-13T16:15:07.000+01:00
Discrepancies between the function returning the Stats struct and the
Metrics() function

Signed-off-by: Eliott Bouhana &lt;eliott.bouhana@datadoghq.com&gt;
diff --git a/metrics.go b/metrics.go
@@ -35,7 +35,7 @@ const (
 )
 
 // Metrics transform the stats returned by the WAF into a map of key value metrics for datadog backend
-func (stats *Stats) Metrics() map[string]any {
+func (stats Stats) Metrics() map[string]any {
 	tags := make(map[string]any, len(stats.Timers)+len(stats.Truncations)+1)
 	for k, v := range stats.Timers {
 		tags[k] = uint64(v.Microseconds())

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ const (`
`35`	`35`	`)`
`36`	`36`
`37`	`37`	`// Metrics transform the stats returned by the WAF into a map of key value metrics for datadog backend`
`38`		`-func (stats *Stats) Metrics() map[string]any {`
	`38`	`+func (stats Stats) Metrics() map[string]any {`
`39`	`39`	`tags := make(map[string]any, len(stats.Timers)+len(stats.Truncations)+1)`
`40`	`40`	`for k, v := range stats.Timers {`
`41`	`41`	`tags[k] = uint64(v.Microseconds())`