ref(core): Move shouldPropagateTraceForUrl from opentelemetry to core (#19254)

This function is needed in an upcoming pr by node-core/light, and since
node-core/light does not depend on OpenTelemetry we need to move this to
core and re-export it in `@sentry/opentelemetry` for backwards
compatibility.

Closes #19255 (added automatically)

Author: andreiborza

packages/core/src/index.ts

@@ -92,6 +92,7 @@ export { _setSpanForScope as _INTERNAL_setSpanForScope } from './utils/spanOnSco
 export { parseSampleRate } from './utils/parseSampleRate';
 export { applySdkMetadata } from './utils/sdkMetadata';
 export { getTraceData } from './utils/traceData';
+export { shouldPropagateTraceForUrl } from './utils/tracePropagationTargets';
 export { getTraceMetaTags } from './utils/meta';
 export { debounce } from './utils/debounce';
 export {

packages/core/src/utils/tracePropagationTargets.ts

@@ -0,0 +1,35 @@
+import { DEBUG_BUILD } from '../debug-build';
+import type { CoreOptions as Options } from '../types-hoist/options';
+import { debug } from './debug-logger';
+import type { LRUMap } from './lru';
+import { stringMatchesSomePattern } from './string';
+
+const NOT_PROPAGATED_MESSAGE =
+  '[Tracing] Not injecting trace data for url because it does not match tracePropagationTargets:';
+
+/**
+ * Check if a given URL should be propagated to or not.
+ * If no url is defined, or no trace propagation targets are defined, this will always return `true`.
+ * You can also optionally provide a decision map, to cache decisions and avoid repeated regex lookups.
+ */
+export function shouldPropagateTraceForUrl(
+  url: string | undefined,
+  tracePropagationTargets: Options['tracePropagationTargets'],
+  decisionMap?: LRUMap<string, boolean>,
+): boolean {
+  if (typeof url !== 'string' || !tracePropagationTargets) {
+    return true;
+  }
+
+  const cachedDecision = decisionMap?.get(url);
+  if (cachedDecision !== undefined) {
+    DEBUG_BUILD && !cachedDecision && debug.log(NOT_PROPAGATED_MESSAGE, url);
+    return cachedDecision;
+  }
+
+  const decision = stringMatchesSomePattern(url, tracePropagationTargets);
+  decisionMap?.set(url, decision);
+
+  DEBUG_BUILD && !decision && debug.log(NOT_PROPAGATED_MESSAGE, url);
+  return decision;
+}

packages/opentelemetry/src/propagator.ts

@@ -2,7 +2,7 @@ import type { Baggage, Context, Span, SpanContext, TextMapGetter, TextMapSetter
 import { context, INVALID_TRACEID, propagation, trace, TraceFlags } from '@opentelemetry/api';
 import { isTracingSuppressed, W3CBaggagePropagator } from '@opentelemetry/core';
 import { ATTR_URL_FULL, SEMATTRS_HTTP_URL } from '@opentelemetry/semantic-conventions';
-import type { Client, continueTrace, DynamicSamplingContext, Options, Scope } from '@sentry/core';
+import type { Client, continueTrace, DynamicSamplingContext, Scope } from '@sentry/core';
 import {
   baggageHeaderToDynamicSamplingContext,
   debug,
@@ -18,8 +18,8 @@ import {
   propagationContextFromHeaders,
   SENTRY_BAGGAGE_KEY_PREFIX,
   shouldContinueTrace,
+  shouldPropagateTraceForUrl,
   spanToJSON,
-  stringMatchesSomePattern,
 } from '@sentry/core';
 import { SENTRY_BAGGAGE_HEADER, SENTRY_TRACE_HEADER, SENTRY_TRACE_STATE_URL } from './constants';
 import { DEBUG_BUILD } from './debug-build';
@@ -124,35 +124,8 @@ export class SentryPropagator extends W3CBaggagePropagator {
   }
 }
 
-const NOT_PROPAGATED_MESSAGE =
-  '[Tracing] Not injecting trace data for url because it does not match tracePropagationTargets:';
-
-/**
- * Check if a given URL should be propagated to or not.
- * If no url is defined, or no trace propagation targets are defined, this will always return `true`.
- * You can also optionally provide a decision map, to cache decisions and avoid repeated regex lookups.
- */
-export function shouldPropagateTraceForUrl(
-  url: string | undefined,
-  tracePropagationTargets: Options['tracePropagationTargets'],
-  decisionMap?: LRUMap<string, boolean>,
-): boolean {
-  if (typeof url !== 'string' || !tracePropagationTargets) {
-    return true;
-  }
-
-  const cachedDecision = decisionMap?.get(url);
-  if (cachedDecision !== undefined) {
-    DEBUG_BUILD && !cachedDecision && debug.log(NOT_PROPAGATED_MESSAGE, url);
-    return cachedDecision;
-  }
-
-  const decision = stringMatchesSomePattern(url, tracePropagationTargets);
-  decisionMap?.set(url, decision);
-
-  DEBUG_BUILD && !decision && debug.log(NOT_PROPAGATED_MESSAGE, url);
-  return decision;
-}
+// Re-exported from @sentry/core for backwards compatibility
+export { shouldPropagateTraceForUrl } from '@sentry/core';
 
 /**
  * Get propagation injection data for the given context.