From 65ca511300fb250098d1dbcddc7d2af419935e49 Mon Sep 17 00:00:00 2001
From: kruskal <99559985+kruskall@users.noreply.github.com>
Date: Sat, 16 Jul 2022 23:14:41 +0200
Subject: [PATCH] fix: do not ignore marshal errors when handling Report log
 event

Avoid sending empty metrics data and return the marshal error
instead of ignoring it.
The caller logs the error with the appropriate log level.
---
 apm-lambda-extension/logsapi/process_metrics.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apm-lambda-extension/logsapi/process_metrics.go b/apm-lambda-extension/logsapi/process_metrics.go
index 7b8c3914..8d0ece91 100644
--- a/apm-lambda-extension/logsapi/process_metrics.go
+++ b/apm-lambda-extension/logsapi/process_metrics.go
@@ -99,7 +99,7 @@ func ProcessPlatformReport(ctx context.Context, metadataContainer *extension.Met
 
 	var jsonWriter fastjson.Writer
 	if err := metricsContainer.MarshalFastJSON(&jsonWriter); err != nil {
-		return extension.AgentData{Data: metricsData}, nil
+		return extension.AgentData{}, err
 	}
 
 	if metadataContainer.Metadata != nil {