Use integers in span attributes again

fiberplane · Sep 16, 2024 · 7622f8b · 7622f8b
1 parent a0e138d
commit 7622f8b
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/packages/client-library-otel/src/utils/request.ts b/packages/client-library-otel/src/utils/request.ts
@@ -250,7 +250,7 @@ export async function getResponseAttributes(
   response: GlobalResponse | HonoResponse,
 ) {
   const attributes: Attributes = {
-    [EXTRA_SEMATTRS_HTTP_RESPONSE_STATUS_CODE]: String(response.status),
+    [EXTRA_SEMATTRS_HTTP_RESPONSE_STATUS_CODE]: response.status,
     [SEMATTRS_HTTP_SCHEME]: response.url.split(":")[0],
   };
 
@@ -261,7 +261,14 @@ export async function getResponseAttributes(
 
   const contentLength = response.headers.get("content-length");
   if (contentLength) {
-    attributes[SEMATTRS_HTTP_RESPONSE_CONTENT_LENGTH] = contentLength;
+    try {
+      attributes[SEMATTRS_HTTP_RESPONSE_CONTENT_LENGTH] = Number.parseInt(
+        contentLength,
+        10,
+      );
+    } catch {
+      // ignore errors
+    }
   }
 
   const headers = response.headers;