[8.11](backport #36956) x-pack/filebeat/input/internal/httplog: fix i…

…ncorrect append source (#36959) * x-pack/filebeat/input/internal/httplog: fix incorrect append source (#36956) (cherry picked from commit bbf0111) * remove irrelevant changelog lines --------- Co-authored-by: Dan Kortschak <[email protected]>
elastic · Oct 25, 2023 · fd0446f · fd0446f
1 parent 93504ea
commit fd0446f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc
@@ -109,6 +109,7 @@ is collected by it.
 - Ensure winlog input retains metric collection when handling recoverable errors. {issue}36479[36479] {pull}36483[36483]
 - Revert error introduced in {pull}35734[35734] when symlinks can't be resolved in filestream. {pull}36557[36557]
 - Fix ignoring external input configuration in `take_over: true` mode {issue}36378[36378] {pull}36395[36395]
+- Fix handling of response errors in HTTPJSON and CEL request trace logging. {pull}36956[36956]
 
 *Heartbeat*
 

diff --git a/x-pack/filebeat/input/internal/httplog/roundtripper.go b/x-pack/filebeat/input/internal/httplog/roundtripper.go
@@ -158,9 +158,9 @@ func (rt *LoggingRoundTripper) RoundTrip(req *http.Request) (*http.Response, err
 	switch len(errorsMessages) {
 	case 0:
 	case 1:
-		respParts = append(reqParts, zap.String("error.message", errorsMessages[0]))
+		respParts = append(respParts, zap.String("error.message", errorsMessages[0]))
 	default:
-		respParts = append(reqParts, zap.Strings("error.message", errorsMessages))
+		respParts = append(respParts, zap.Strings("error.message", errorsMessages))
 	}
 	log.Debug("HTTP response", respParts...)