Skip to content

Commit d19187d

Browse files
mustakimalilalitbcijothomas
authored
chore: Add error logging for failed OpenTelemetry HTTP export (#1498)
Co-authored-by: Lalit Kumar Bhasin <labhas@microsoft.com> Co-authored-by: Cijo Thomas <cijo.thomas@gmail.com>
1 parent 57c3aa3 commit d19187d

File tree

2 files changed

+24
-2
lines changed

2 files changed

+24
-2
lines changed

opentelemetry-otlp/src/exporter/http/logs.rs

+12-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,18 @@ impl LogExporter for OtlpHttpClient {
3131
request.headers_mut().insert(k.clone(), v.clone());
3232
}
3333

34-
client.send(request).await?;
34+
let request_uri = request.uri().to_string();
35+
let response = client.send(request).await?;
36+
37+
if !response.status().is_success() {
38+
let error = format!(
39+
"OpenTelemetry logs export failed. Url: {}, Status Code: {}, Response: {:?}",
40+
response.status().as_u16(),
41+
request_uri,
42+
response.body()
43+
);
44+
return Err(LogError::Other(error.into()));
45+
}
3546

3647
Ok(())
3748
}

opentelemetry-otlp/src/exporter/http/trace.rs

+12-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,18 @@ impl SpanExporter for OtlpHttpClient {
4646
}
4747

4848
Box::pin(async move {
49-
client.send(request).await?;
49+
let request_uri = request.uri().to_string();
50+
let response = client.send(request).await?;
51+
52+
if !response.status().is_success() {
53+
let error = format!(
54+
"OpenTelemetry trace export failed. Url: {}, Status Code: {}, Response: {:?}",
55+
response.status().as_u16(),
56+
request_uri,
57+
response.body()
58+
);
59+
return Err(TraceError::Other(error.into()));
60+
}
5061

5162
Ok(())
5263
})

0 commit comments

Comments
 (0)