no debugonly section

open-telemetry · Jan 31, 2024 · 5fd5a9f · 5fd5a9f
1 parent f416b28
commit 5fd5a9f
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 30 deletions.
diff --git a/opentelemetry-otlp/src/exporter/http/logs.rs b/opentelemetry-otlp/src/exporter/http/logs.rs
@@ -31,22 +31,17 @@ impl LogExporter for OtlpHttpClient {
             request.headers_mut().insert(k.clone(), v.clone());
         }
 
-        #[cfg(not(debug_assertions))]
-        client.send(request).await?;
+        let request_uri = request.uri().to_string();
+        let response = client.send(request).await?;
 
-        #[cfg(debug_assertions)]
-        {
-            let request_uri = request.uri().to_string();
-            let response = client.send(request).await?;
-
-            if !response.status().is_success() {
-                let error = format!(
-                    "OpenTelemetry logs export failed. Url: {}, Response: {:?}",
-                    request_uri,
-                    response.body()
-                );
-                return Err(LogError::Other(error.into()));
-            }
+        if !response.status().is_success() {
+            let error = format!(
+                "OpenTelemetry logs export failed. Url: {}, Status Code: {}, Response: {:?}",
+                response.status().as_u16(),
+                request_uri,
+                response.body()
+            );
+            return Err(LogError::Other(error.into()));
         }
 
         Ok(())

diff --git a/opentelemetry-otlp/src/exporter/http/trace.rs b/opentelemetry-otlp/src/exporter/http/trace.rs
@@ -46,22 +46,17 @@ impl SpanExporter for OtlpHttpClient {
         }
 
         Box::pin(async move {
-            #[cfg(not(debug_assertions))]
-            client.send(request).await?;
+            let request_uri = request.uri().to_string();
+            let response = client.send(request).await?;
 
-            #[cfg(debug_assertions)]
-            {
-                let request_uri = request.uri().to_string();
-                let response = client.send(request).await?;
-
-                if !response.status().is_success() {
-                    let error = format!(
-                        "OpenTelemetry trace export failed. Url: {}, Response: {:?}",
-                        request_uri,
-                        response.body()
-                    );
-                    return Err(TraceError::Other(error.into()));
-                }
+            if !response.status().is_success() {
+                let error = format!(
+                    "OpenTelemetry trace export failed. Url: {}, Status Code: {}, Response: {:?}",
+                    response.status().as_u16(),
+                    request_uri,
+                    response.body()
+                );
+                return Err(TraceError::Other(error.into()));
             }
 
             Ok(())