Skip to content

Commit b753803

Browse files
Rishikesh1159meetvm
authored and
meetvm
committed
Add Response Status Number in http trace logs. (opensearch-project#16978)
Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>
1 parent 4a5c5c3 commit b753803

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

server/src/main/java/org/opensearch/http/HttpTracer.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -116,10 +116,11 @@ void traceResponse(
116116
) {
117117
logger.trace(
118118
new ParameterizedMessage(
119-
"[{}][{}][{}][{}][{}] sent response to [{}] success [{}]",
119+
"[{}][{}][{}][{}][{}][{}] sent response to [{}] success [{}]",
120120
requestId,
121121
opaqueHeader,
122122
restResponse.status(),
123+
restResponse.status().getStatus(),
123124
restResponse.contentType(),
124125
contentLength,
125126
httpChannel,

server/src/test/java/org/opensearch/http/AbstractHttpServerTransportTests.java

+2
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,8 @@ public HttpStats stats() {
285285
+ opaqueId
286286
+ "\\]\\["
287287
+ (badRequest ? "BAD_REQUEST" : "OK")
288+
+ "\\]\\["
289+
+ (badRequest ? "400" : "200")
288290
+ "\\]\\[null\\]\\[0\\] sent response to \\[.*"
289291
)
290292
);

0 commit comments

Comments
 (0)