enhance: Check if resp is nil in capi metrics and continue

crowdsecurity · Oct 25, 2024 · 691ecfb · 691ecfb
1 parent d8bc17b
commit 691ecfb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/apiserver/apic_metrics.go b/pkg/apiserver/apic_metrics.go
@@ -368,9 +368,10 @@ func (a *apic) SendUsageMetrics(ctx context.Context) {
 			if err != nil {
 				log.Errorf("unable to send usage metrics: %s", err)
 
-				if resp.Response.StatusCode >= http.StatusBadRequest && resp.Response.StatusCode != http.StatusUnprocessableEntity {
+				if resp == nil || resp.Response.StatusCode >= http.StatusBadRequest && resp.Response.StatusCode != http.StatusUnprocessableEntity {
 					// In case of 422, mark the metrics as sent anyway, the API did not like what we sent,
 					// and it's unlikely we'll be able to fix it
+					// also if resp is nil, we should'nt mark the metrics as sent could be network issue
 					continue
 				}
 			}