Use integers in span attributes again

fiberplane · Aug 29, 2024 · 0e92b2c · 0e92b2c
1 parent c31e14a
commit 0e92b2c
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
@@ -237,7 +237,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],
   };
 
@@ -248,7 +248,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;