@pingops/otel 0.2.2 → 0.2.4

package/dist/index.cjs

@@ -60,6 +60,21 @@ function getGlobalConfig() {
 //#endregion
 //#region src/span-processor.ts
 const logger$2 = (0, _pingops_core.createLogger)("[PingOps Processor]");
+function normalizePath$1(pathname) {
+	return pathname.replace(/\/+/g, "/").replace(/\/$/, "");
+}
+function isExporterRequestUrl$1(url$1, exporterUrl) {
+	try {
+		const request = new URL(url$1);
+		const exporter = new URL(exporterUrl);
+		if (request.origin !== exporter.origin) return false;
+		const requestPath = normalizePath$1(request.pathname);
+		const exporterPath = normalizePath$1(exporter.pathname);
+		return requestPath === exporterPath || requestPath.startsWith(`${exporterPath}/`);
+	} catch {
+		return false;
+	}
+}
 /**
 * Creates a filtered span wrapper that applies header filtering to attributes
 *
@@ -93,6 +108,7 @@ function createFilteredSpan(span, domainAllowList, globalHeadersAllowList, globa
 */
 var PingopsSpanProcessor = class {
 	processor;
+	exporterTraceUrl;
 	config;
 	/**
 	* Creates a new PingopsSpanProcessor instance.
@@ -102,8 +118,9 @@ var PingopsSpanProcessor = class {
 	constructor(config) {
 		const exportMode = config.exportMode ?? "batched";
 		const apiKey = config.apiKey || process.env.PINGOPS_API_KEY || "";
+		this.exporterTraceUrl = `${config.baseUrl}/v1/traces`;
 		const exporter = new _opentelemetry_exporter_trace_otlp_http.OTLPTraceExporter({
-			url: `${config.baseUrl}/v1/traces`,
+			url: this.exporterTraceUrl,
 			headers: {
 				Authorization: apiKey ? `Bearer ${apiKey}` : "",
 				"Content-Type": "application/json"
@@ -131,7 +148,7 @@ var PingopsSpanProcessor = class {
 			domainAllowList: config.domainAllowList,
 			maxRequestBodySize: config.maxRequestBodySize,
 			maxResponseBodySize: config.maxResponseBodySize,
-			exportTraceUrl: `${config.baseUrl}/v1/traces`
+			exportTraceUrl: this.exporterTraceUrl
 		});
 		logger$2.info("Initialized PingopsSpanProcessor", {
 			baseUrl: config.baseUrl,
@@ -192,6 +209,14 @@ var PingopsSpanProcessor = class {
 			}
 			const attributes = span.attributes;
 			const url$1 = (0, _pingops_core.getHttpUrlFromAttributes)(attributes) ?? "";
+			if (url$1 && isExporterRequestUrl$1(url$1, this.exporterTraceUrl)) {
+				logger$2.debug("Skipping exporter span to prevent self-instrumentation", {
+					spanName: span.name,
+					spanId: spanContext.spanId,
+					url: url$1
+				});
+				return;
+			}
 			logger$2.debug("Extracted URL for domain filtering", {
 				spanName: span.name,
 				url: url$1,
@@ -363,9 +388,12 @@ async function shutdownTracerProvider() {
 //#region src/instrumentations/suppression-guard.ts
 const logger = (0, _pingops_core.createLogger)("[PingOps SuppressionGuard]");
 let hasLoggedSuppressionLeakWarning = false;
-function normalizeUrl(url$1) {
+function normalizePath(pathname) {
+	return pathname.replace(/\/+/g, "/").replace(/\/$/, "");
+}
+function parseUrl(url$1) {
 	try {
-		return new URL(url$1).toString();
+		return new URL(url$1);
 	} catch {
 		return null;
 	}
@@ -374,10 +402,20 @@ function isExporterRequestUrl(requestUrl) {
 	if (!requestUrl) return false;
 	const exporterUrl = getGlobalConfig()?.exportTraceUrl;
 	if (!exporterUrl) return false;
-	const normalizedRequestUrl = normalizeUrl(requestUrl);
-	const normalizedExporterUrl = normalizeUrl(exporterUrl);
-	if (!normalizedRequestUrl || !normalizedExporterUrl) return false;
-	return normalizedRequestUrl.startsWith(normalizedExporterUrl);
+	const parsedRequestUrl = parseUrl(requestUrl);
+	const parsedExporterUrl = parseUrl(exporterUrl);
+	if (!parsedRequestUrl || !parsedExporterUrl) return false;
+	if (parsedRequestUrl.origin !== parsedExporterUrl.origin) return false;
+	const requestPath = normalizePath(parsedRequestUrl.pathname);
+	const exporterPath = normalizePath(parsedExporterUrl.pathname);
+	return requestPath === exporterPath || requestPath.startsWith(`${exporterPath}/`);
+}
+/**
+* Determines whether an outbound request should be skipped from instrumentation
+* to prevent exporter self-instrumentation loops.
+*/
+function shouldIgnoreOutboundInstrumentation(requestUrl) {
+	return isExporterRequestUrl(requestUrl);
 }
 /**
 * Returns a context for outbound span creation that neutralizes leaked suppression
@@ -555,6 +593,16 @@ function captureRequestHeaders(span, headers) {
 function captureResponseHeaders(span, headers) {
 	for (const [key, value] of Object.entries(headers)) if (value !== void 0) span.setAttribute(`http.response.header.${key.toLowerCase()}`, Array.isArray(value) ? value.join(",") : String(value));
 }
+function extractRequestUrlFromSpanOptions(options) {
+	const attrs = options.attributes;
+	if (!attrs) return;
+	if (typeof attrs[_opentelemetry_semantic_conventions.ATTR_URL_FULL] === "string") return attrs[_opentelemetry_semantic_conventions.ATTR_URL_FULL];
+	if (typeof attrs[_opentelemetry_semantic_conventions.ATTR_HTTP_URL] === "string") return attrs[_opentelemetry_semantic_conventions.ATTR_HTTP_URL];
+	const scheme = typeof attrs[_opentelemetry_semantic_conventions.ATTR_URL_SCHEME] === "string" ? attrs[_opentelemetry_semantic_conventions.ATTR_URL_SCHEME] : "http";
+	const host = typeof attrs[_opentelemetry_semantic_conventions.ATTR_SERVER_ADDRESS] === "string" ? attrs[_opentelemetry_semantic_conventions.ATTR_SERVER_ADDRESS] : void 0;
+	if (!host) return;
+	return `${scheme}://${host}${typeof attrs[_opentelemetry_semantic_conventions.ATTR_SERVER_PORT] === "number" ? `:${attrs[_opentelemetry_semantic_conventions.ATTR_SERVER_PORT]}` : ""}${typeof attrs[_opentelemetry_semantic_conventions.ATTR_URL_PATH] === "string" ? attrs[_opentelemetry_semantic_conventions.ATTR_URL_PATH] : "/"}${typeof attrs[_opentelemetry_semantic_conventions.ATTR_URL_QUERY] === "string" && attrs[_opentelemetry_semantic_conventions.ATTR_URL_QUERY].length > 0 ? `?${attrs[_opentelemetry_semantic_conventions.ATTR_URL_QUERY]}` : ""}`;
+}
 const PingopsHttpSemanticAttributes = PingopsSemanticAttributes;
 var PingopsHttpInstrumentation = class extends _opentelemetry_instrumentation_http.HttpInstrumentation {
 	constructor(config) {
@@ -572,7 +620,7 @@ var PingopsHttpInstrumentation = class extends _opentelemetry_instrumentation_ht
 		const originalStartHttpSpan = target._startHttpSpan.bind(this);
 		target._startHttpSpan = (name, options, ctx = _opentelemetry_api.context.active()) => {
 			if (options.kind !== _opentelemetry_api.SpanKind.CLIENT) return originalStartHttpSpan(name, options, ctx);
-			return originalStartHttpSpan(name, options, resolveOutboundSpanParentContext(ctx, typeof options.attributes?.["url.full"] === "string" ? options.attributes["url.full"] : void 0));
+			return originalStartHttpSpan(name, options, resolveOutboundSpanParentContext(ctx, extractRequestUrlFromSpanOptions(options)));
 		};
 	}
 	_createConfig(config) {
@@ -635,6 +683,14 @@ var PingopsHttpInstrumentation = class extends _opentelemetry_instrumentation_ht
 /**
 * HTTP instrumentation for OpenTelemetry
 */
+function toRequestUrl$1(request) {
+	if (typeof request.href === "string" && request.href.length > 0) return request.href;
+	const protocol = typeof request.protocol === "string" && request.protocol.length > 0 ? request.protocol : "http:";
+	const hostnameOrHost = typeof request.hostname === "string" && request.hostname.length > 0 ? request.hostname : request.host;
+	if (!hostnameOrHost) return;
+	const hasPortInHost = hostnameOrHost.includes(":");
+	return `${protocol}//${hostnameOrHost}${request.port != null && !hasPortInHost ? `:${request.port}` : ""}${typeof request.path === "string" ? request.path : typeof request.pathname === "string" ? request.pathname : "/"}`;
+}
 /**
 * Creates an HTTP instrumentation instance.
 * All outgoing HTTP requests are instrumented when the SDK is initialized.
@@ -644,12 +700,16 @@ var PingopsHttpInstrumentation = class extends _opentelemetry_instrumentation_ht
 */
 function createHttpInstrumentation(config) {
 	const globalConfig$1 = getGlobalConfig();
+	const userIgnoreOutgoingRequestHook = config?.ignoreOutgoingRequestHook;
 	return new PingopsHttpInstrumentation({
+		...config,
 		ignoreIncomingRequestHook: () => true,
-		ignoreOutgoingRequestHook: () => false,
+		ignoreOutgoingRequestHook: (request) => {
+			if (shouldIgnoreOutboundInstrumentation(toRequestUrl$1(request))) return true;
+			return userIgnoreOutgoingRequestHook?.(request) ?? false;
+		},
 		maxRequestBodySize: globalConfig$1?.maxRequestBodySize,
-		maxResponseBodySize: globalConfig$1?.maxResponseBodySize,
-		...config
+		maxResponseBodySize: globalConfig$1?.maxResponseBodySize
 	});
 }
 
@@ -1028,6 +1088,13 @@ var UndiciInstrumentation = class extends _opentelemetry_instrumentation.Instrum
 /**
 * Undici instrumentation for OpenTelemetry
 */
+function toRequestUrl(request) {
+	try {
+		return new URL(request.path, request.origin).toString();
+	} catch {
+		return;
+	}
+}
 /**
 * Creates an Undici instrumentation instance.
 * All requests are instrumented when the SDK is initialized.
@@ -1038,7 +1105,9 @@ function createUndiciInstrumentation() {
 	const globalConfig$1 = getGlobalConfig();
 	return new UndiciInstrumentation({
 		enabled: true,
-		ignoreRequestHook: () => false,
+		ignoreRequestHook: (request) => {
+			return shouldIgnoreOutboundInstrumentation(toRequestUrl(request));
+		},
 		maxRequestBodySize: globalConfig$1?.maxRequestBodySize,
 		maxResponseBodySize: globalConfig$1?.maxResponseBodySize
 	});