From d3cc39308690238071a105e8b58fac02e2de0646 Mon Sep 17 00:00:00 2001
From: Yash Golani <yash.golani.ctr@sumologic.com>
Date: Sat, 7 Dec 2024 17:20:32 +0530
Subject: [PATCH 1/2] TRACING-4435 fixed

---
 src/index.ts                          |  1 +
 src/sumologic-span-processor/index.ts | 31 +++++++++++++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/src/index.ts b/src/index.ts
index 7fcdfb0..e0f6a22 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -195,6 +195,7 @@ export const initialize = ({
       dropSingleUserInteractionTraces,
       getOverriddenServiceName,
       defaultServiceName,
+      ignoreUrls,
     }),
   );
 
diff --git a/src/sumologic-span-processor/index.ts b/src/sumologic-span-processor/index.ts
index c9d0b90..eb0d1ba 100644
--- a/src/sumologic-span-processor/index.ts
+++ b/src/sumologic-span-processor/index.ts
@@ -20,6 +20,7 @@ export interface SumoLogicSpanProcessorConfig
   dropSingleUserInteractionTraces?: boolean;
   getOverriddenServiceName?: (span: SdkTraceSpan) => string;
   defaultServiceName: string;
+  ignoreUrls?: (string | RegExp)[]; // Allow both strings and RegExp
 }
 
 export class SumoLogicSpanProcessor extends BatchSpanProcessor {
@@ -31,6 +32,8 @@ export class SumoLogicSpanProcessor extends BatchSpanProcessor {
 
   private traceProcessor: ReturnType<typeof createTraceProcessor>;
 
+  private static ignoreUrls: RegExp[] = [/youtube\.com/];
+
   constructor(exporter: SpanExporter, config: SumoLogicSpanProcessorConfig) {
     super(exporter, config);
     this.shouldCollectSessionId = config.collectSessionId ?? true;
@@ -41,9 +44,31 @@ export class SumoLogicSpanProcessor extends BatchSpanProcessor {
     this.defaultServiceName = config.defaultServiceName;
 
     this.traceProcessor = createTraceProcessor(this);
+    if (config.ignoreUrls) {
+      const normalizedUrls = config.ignoreUrls.map((url) =>
+        typeof url === 'string'
+          ? new RegExp(url.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'))
+          : url,
+      );
+      SumoLogicSpanProcessor.ignoreUrls.push(...normalizedUrls);
+    }
+  }
+
+  private shouldIgnoreSpan(span: SdkTraceSpan | ReadableSpan): boolean {
+    const url = span.attributes['http.url'] as string;
+    if (!url) return false;
+    return SumoLogicSpanProcessor.ignoreUrls.some((pattern) =>
+      pattern.test(url),
+    );
   }
 
   onStart(span: SdkTraceSpan, context: Context = ROOT_CONTEXT): void {
+    // Check if the span should be ignored
+    if (this.shouldIgnoreSpan(span)) {
+      // Skip processing this span
+      return;
+    }
+
     documentVisibilityState.onStart(span, context);
     rootToChildEnrichment.onStart(span, context);
     this.traceProcessor.onStart(span, context);
@@ -62,6 +87,12 @@ export class SumoLogicSpanProcessor extends BatchSpanProcessor {
   }
 
   onEnd(span: ReadableSpan): void {
+    // Check if the span should be ignored
+    if (this.shouldIgnoreSpan(span)) {
+      // Skip processing this span
+      return;
+    }
+
     documentVisibilityState.onEnd(span);
 
     // we use callbacks instead of Promises, because even immediately resolved Promise won't be executed synchronously

From 4b4490101d505cb78bec4f28e080763d6c29130b Mon Sep 17 00:00:00 2001
From: Yash Golani <yash.golani.ctr@sumologic.com>
Date: Mon, 9 Dec 2024 11:40:00 +0530
Subject: [PATCH 2/2] TRACING-4435 refactored

---
 src/sumologic-span-processor/index.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/sumologic-span-processor/index.ts b/src/sumologic-span-processor/index.ts
index eb0d1ba..815713e 100644
--- a/src/sumologic-span-processor/index.ts
+++ b/src/sumologic-span-processor/index.ts
@@ -32,7 +32,7 @@ export class SumoLogicSpanProcessor extends BatchSpanProcessor {
 
   private traceProcessor: ReturnType<typeof createTraceProcessor>;
 
-  private static ignoreUrls: RegExp[] = [/youtube\.com/];
+  private static ignoreUrls: RegExp[] = [];
 
   constructor(exporter: SpanExporter, config: SumoLogicSpanProcessorConfig) {
     super(exporter, config);