@dash0/sdk-web 0.16.1 → 0.16.3

package/src/instrumentations/http/fetch.ts

@@ -1,4 +1,14 @@
-import { debug, observeResourcePerformance, perf, win, setTimeout, isSameOrigin, wrap, parseUrl } from "../../utils";
+import {
+  debug,
+  observeResourcePerformance,
+  perf,
+  win,
+  setTimeout,
+  isSameOrigin,
+  wrap,
+  parseUrl,
+  clearTimeout,
+} from "../../utils";
 import { isUrlIgnored, matchesAny } from "../../utils/ignore-rules";
 import {
   addAttribute,
@@ -156,6 +166,8 @@ function isGraphQLQuery(input: RequestInfo | URL, init?: RequestInit) {
 
 function tryCaptureHttpHeaders(headers: Headers, span: InProgressSpan, getAttributeKey: (headerKey: string) => string) {
   try {
+    if (!vars.headersToCapture.length) return;
+
     headers.forEach((value, key) => {
       if (vars.headersToCapture.some((rxp) => rxp.test(key))) {
         addAttribute(span.attributes, getAttributeKey(key), value);
@@ -194,7 +206,8 @@ function wrapResponse(
   let fallbackTs: number = perf.now();
   const body = originalResponse.body;
 
-  if (!body) {
+  // For some reason browsers return a response body on responses that can't be constructed with one. In that case we can't wrap the body stream
+  if (!body || !responseCanHaveBody(originalResponse)) {
     onDone();
     return originalResponse;
   }
@@ -314,3 +327,8 @@ function addTraceContextHttpHeaders(
     }
   }
 }
+
+function responseCanHaveBody(response: Response) {
+  const status = response.status;
+  return status >= 200 && status != 204 && status != 205 && status != 304;
+}

package/src/transport/fetch.ts

@@ -12,7 +12,7 @@ export async function send(path: string, body: unknown): Promise<void> {
   let isCompressed = false;
 
   // Try to compress if supported
-  if (typeof CompressionStream !== "undefined") {
+  if (typeof CompressionStream !== "undefined" && vars.enableTransportCompression) {
     requestBody = await compressWithGzip(jsonString);
     byteLength = requestBody.byteLength;
     isCompressed = true;

package/src/types/options.ts

@@ -58,5 +58,6 @@ export type InitOptions = {
     | "headersToCapture"
     | "urlAttributeScrubber"
     | "pageViewInstrumentation"
+    | "enableTransportCompression"
   >
 >;

package/src/utils/wrap.ts

@@ -26,6 +26,7 @@ export function wrap<ModuleType extends object, TargetNameType extends keyof Mod
 
   if (isAlreadyInstrumented(original)) {
     debug(`${String(target)} has already been instrumented, skipping`);
+    return;
   }
 
   markAsInstrumented(original);

package/src/vars.ts

@@ -165,6 +165,12 @@ export type Vars = {
   urlAttributeScrubber: UrlAttributeScrubber;
 
   pageViewInstrumentation: PageViewInstrumentationSettings;
+
+  /**
+   * Enables telemetry transport compression using gzip.
+   * experimental - in rare cases causes Chrome to crash to use at your own risk.
+   */
+  enableTransportCompression: boolean;
 };
 
 export const vars: Vars = {
@@ -191,4 +197,5 @@ export const vars: Vars = {
     trackVirtualPageViews: true,
     includeParts: [],
   },
+  enableTransportCompression: false,
 };