fix(opentelemetry): Ensure unsent spans are ignored in span exporter

Author: mydea

packages/opentelemetry/src/spanExporter.ts

@@ -180,9 +180,6 @@ export class SentrySpanExporter {
 
     this.flushSentSpanCache();
     const sentSpans = this._maybeSend(finishedSpans);
-    for (const span of finishedSpans) {
-      this._sentSpans.set(span.spanContext().spanId, Date.now() + DEFAULT_TIMEOUT * 1000);
-    }
 
     const sentSpanCount = sentSpans.size;
     const remainingOpenSpanCount = finishedSpans.length - sentSpanCount;
@@ -192,6 +189,7 @@ export class SentrySpanExporter {
       );
 
     for (const span of sentSpans) {
+      this._sentSpans.set(span.spanContext().spanId, Date.now() + DEFAULT_TIMEOUT * 1000);
       const bucketEntry = this._spansToBucketEntry.get(span);
       if (bucketEntry) {
         bucketEntry.spans.delete(span);