@maple-dev/browser 0.10.0 → 0.10.1

package/README.md

@@ -68,8 +68,9 @@ fingerprint to one contentless issue that buries the real ones. Add
 
 ### Failed HTTP requests
 
-`errors: { captureHttpStatus: [[500, 599]] }` makes those `fetch`/XHR responses errors, typed by
-status. By default a response status alone is not an error; a network failure always is.
+`fetch`/XHR responses with a 4xx or 5xx status are errors, typed by status, as the HTTP semantic
+conventions say for client spans. Narrow that with `errors: { captureHttpStatus: [[500, 599]] }`.
+A network failure is always an error.
 
 ### Breadcrumbs
 

package/dist/{deferred-CdtlxG0Q.mjs → deferred-BDp5NC7U.mjs}

@@ -1,5 +1,5 @@
 import { E as selectorOf, F as scrubUrl, l as onSessionEvent, w as sdkHint, x as ingestHeaders } from "./sink-BOR9_z4b.mjs";
-import { _ as consentRevokedAt, a as detachLogSink, c as resourceAttributes, d as attachSpanStash, f as claimPageSignal, g as consentAllowedSince, h as severityOf, i as attachLogSink, l as SDK_NAME, m as Severity, n as navigationSpanAt, o as emitLog, p as onErrorRecorded, r as onDocumentPageload, s as liveMapleTracer, u as SDK_VERSION, v as hasConsent, y as onConsentChange } from "./src-Bzmp8rVr.mjs";
+import { _ as consentRevokedAt, a as detachLogSink, c as resourceAttributes, d as attachSpanStash, f as claimPageSignal, g as consentAllowedSince, h as severityOf, i as attachLogSink, l as SDK_NAME, m as Severity, n as navigationSpanAt, o as emitLog, p as onErrorRecorded, r as onDocumentPageload, s as liveMapleTracer, u as SDK_VERSION, v as hasConsent, y as onConsentChange } from "./src-D4e1o0x4.mjs";
 import { t as installConsoleCapture } from "./console-BrjBnOiX.mjs";
 import { ROOT_CONTEXT, TraceFlags, context, trace } from "@opentelemetry/api";
 import { resourceFromAttributes } from "@opentelemetry/resources";

package/dist/index.d.mts

@@ -57,8 +57,9 @@ interface ErrorFilterOptions {
   readonly beforeCapture?: (error: Error, hint: ErrorFilterHint) => boolean;
   /**
    * HTTP response statuses that make a client request span an error (and so an
-   * issue), e.g. `[[500, 599]]`. Default none: a response status alone is not
-   * an error, and a network failure always is.
+   * issue). Default `[[400, 599]]`, as the HTTP semantic conventions say for
+   * client spans; narrow it, e.g. `[[500, 599]]`, to count fewer. A network
+   * failure is always an error.
    */
   readonly captureHttpStatus?: ReadonlyArray<HttpStatusRange>;
   /**

package/dist/index.mjs

@@ -1,2 +1,2 @@
-import { t as MapleBrowser } from "./src-Bzmp8rVr.mjs";
+import { t as MapleBrowser } from "./src-D4e1o0x4.mjs";
 export { MapleBrowser };

package/dist/react.mjs

@@ -1,4 +1,4 @@
-import { t as MapleBrowser } from "./src-Bzmp8rVr.mjs";
+import { t as MapleBrowser } from "./src-D4e1o0x4.mjs";
 import { Component } from "react";
 //#region src/react.ts
 /** Component stacks can run to hundreds of lines; the top of it names the failing tree. */

package/dist/{src-Bzmp8rVr.mjs → src-D4e1o0x4.mjs}

@@ -542,6 +542,8 @@ function resolveHeaderCapture(raw) {
 }
 //#endregion
 //#region ../sdk-core/src/http-status.ts
+/** The statuses the HTTP semantic conventions make a client span Error for: every 4xx and 5xx. */
+const DEFAULT_ERROR_STATUS = [[400, 599]];
 const inStatusRanges = (status, ranges) => ranges.some((range) => typeof range === "number" ? range === status : status >= range[0] && status <= range[1]);
 /** The status on a span's attributes, current or pre-1.23 semconv key. */
 function responseStatus(read) {
@@ -550,17 +552,12 @@ function responseStatus(read) {
 	if (typeof value === "string" && /^\d{3}$/.test(value)) return Number(value);
 }
 /**
-* The `error.type` / `error.message` pair for a listed status or a network
-* failure (`TypeError`, `timeout`), e.g. `POST https://api.example.com/users/42 -> 503`.
-* The query is dropped; ids in the path are redacted by the issue fingerprint.
+* The `error.type` for a counted status or a network failure (`TypeError`,
+* `timeout`), per the HTTP semantic conventions. No message: `error.message` is
+* deprecated, and the status code already says what went wrong.
 */
-function httpStatusError(read, status) {
-	const method = read("http.request.method") ?? read("http.method") ?? "GET";
-	const url = String(read("url.full") ?? read("http.url") ?? "").replace(/[?#].*$/, "");
-	return {
-		"error.type": String(status),
-		"error.message": `${String(method)} ${url} -> ${status}`
-	};
+function httpErrorType(status) {
+	return { "error.type": String(status) };
 }
 //#endregion
 //#region ../sdk-core/src/log-record.ts
@@ -842,7 +839,7 @@ function withStatus(span, status, attributes) {
 var HttpStatusExporter = class {
 	inner;
 	captureStatus;
-	constructor(inner, captureStatus = []) {
+	constructor(inner, captureStatus = DEFAULT_ERROR_STATUS) {
 		this.inner = inner;
 		this.captureStatus = captureStatus;
 	}
@@ -853,14 +850,9 @@ var HttpStatusExporter = class {
 			if (span.events.some((event) => event.name === "exception")) return span;
 			return withStatus(span, { code: SpanStatusCode.ERROR }, {
 				...span.attributes,
-				...httpStatusError(read, status)
+				...httpErrorType(status)
 			});
 		}
-		const errorType = span.attributes["error.type"];
-		if (span.kind === SpanKind.CLIENT && span.status.code === SpanStatusCode.ERROR && typeof errorType === "string" && !HTTP_STATUS.test(errorType) && span.attributes["error.message"] === void 0 && !span.events.some((event) => event.name === "exception")) return withStatus(span, span.status, {
-			...span.attributes,
-			"error.message": httpStatusError(read, errorType)["error.message"]
-		});
 		if (!isStatusOnlyError(span)) return span;
 		const { "error.type": _errorType, ...attributes } = span.attributes;
 		return withStatus(span, { code: SpanStatusCode.UNSET }, attributes);
@@ -914,7 +906,7 @@ var OfflineSpanExporter = class {
 };
 //#endregion
 //#region src/version.ts
-const SDK_VERSION = "0.9.0";
+const SDK_VERSION = "0.10.1";
 /** The `x-maple-sdk` value this build sends. */
 const SDK_NAME = "maple-browser";
 //#endregion
@@ -1082,7 +1074,7 @@ function setupTracing(config) {
 			const sent = request instanceof Request ? request.headers : new Headers(request.headers);
 			setHeaderAttributes(span, "request", headers.request, (name) => sent.get(name));
 			if (result instanceof Response) setHeaderAttributes(span, "response", headers.response, (name) => result.headers.get(name));
-			else if (result instanceof Error && result.name !== "AbortError" && !request.signal?.aborted) {
+			else if (result instanceof Error && (isTimeout(result, request.signal) || result.name !== "AbortError" && !request.signal?.aborted)) {
 				span.setStatus({
 					code: SpanStatusCode.ERROR,
 					message: result.message
@@ -1118,6 +1110,11 @@ function setupTracing(config) {
 		}
 	};
 }
+/** `AbortSignal.timeout()` aborts with, and rejects the fetch with, a `TimeoutError` DOMException. */
+function isTimeout(error, signal) {
+	const reason = signal?.reason;
+	return error.name === "TimeoutError" || reason instanceof DOMException && reason.name === "TimeoutError";
+}
 function escapeRegExp(value) {
 	return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
@@ -1533,7 +1530,7 @@ function init(rawConfig) {
 		if (config.tracingEnabled && config.tracingCaptureErrors && !stopErrorCapture) stopErrorCapture = setupErrorCapture();
 		if (!deferredPending) {
 			setLogIdentity(() => activeConfig?.identity?.id);
-			deferredPending = import("./deferred-CdtlxG0Q.mjs").then(({ startDeferred }) => {
+			deferredPending = import("./deferred-BDp5NC7U.mjs").then(({ startDeferred }) => {
 				stopDeferred = startDeferred(config);
 			}).catch(() => {});
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@maple-dev/browser",
-	"version": "0.10.0",
+	"version": "0.10.1",
 	"description": "Maple browser SDK — OpenTelemetry tracing and rrweb session replay in one package. Every span and replay event shares a session id for trace↔replay correlation.",
 	"keywords": [
 		"maple",
@@ -82,4 +82,4 @@
 			"optional": true
 		}
 	}
-}
+}