From 404a7b57b47f65bd6ae4640e16aa31c93b46ddf3 Mon Sep 17 00:00:00 2001 From: Dave Protasowski Date: Wed, 2 Jul 2025 11:40:08 -0400 Subject: [PATCH] add some bounds checks --- observability/metrics/k8s/tools.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/observability/metrics/k8s/tools.go b/observability/metrics/k8s/tools.go index 521c86415..315534bbe 100644 --- a/observability/metrics/k8s/tools.go +++ b/observability/metrics/k8s/tools.go @@ -101,7 +101,7 @@ func (l *latency) Observe(ctx context.Context, verb string, u url.URL, latency t if portStr := u.Port(); portStr != "" { port, err := strconv.ParseInt(portStr, 10, 64) - if err != nil && port > 0 { + if err != nil && port > 0 && port < 65535 { serverPort = int(port) } } @@ -126,7 +126,7 @@ func (r *result) Increment(ctx context.Context, code string, method string, host if code == "200" { // happy path - noop - } else if c, err := strconv.ParseInt(code, 10, 64); err != nil { + } else if c, err := strconv.ParseInt(code, 10, 64); err != nil && c >= 100 && c < 600 { codeInt = int(c) }