WIP debug

Author: mydea

dev-packages/browser-integration-tests/suites/tracing/trace-lifetime/init.js

@@ -7,4 +7,5 @@ Sentry.init({
   integrations: [Sentry.browserTracingIntegration(), Sentry.feedbackIntegration()],
   tracePropagationTargets: ['http://example.com'],
   tracesSampleRate: 1,
+  debug: true,
 });

dev-packages/browser-integration-tests/suites/tracing/trace-lifetime/navigation/test.ts

@@ -69,11 +69,21 @@ sentryTest('creates a new trace on each navigation', async ({ getLocalTestUrl, p
   expect(navigation1TraceContext?.trace_id).not.toEqual(navigation2TraceContext?.trace_id);
 });
 
-sentryTest('error after navigation has navigation traceId', async ({ getLocalTestUrl, page }) => {
+sentryTest.only('error after navigation has navigation traceId', async ({ getLocalTestUrl, page }) => {
   if (shouldSkipTracingTest()) {
     sentryTest.skip();
   }
 
+  await page.route('https://dsn.ingest.sentry.io/**/*', route => {
+    return route.fulfill({
+      status: 200,
+      contentType: 'application/json',
+      body: JSON.stringify({ id: 'test-id' }),
+    });
+  });
+
+  page.on('console', msg => console.log(msg.text()));
+
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   // ensure pageload transaction is finished

packages/core/src/baseclient.ts

@@ -709,6 +709,8 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
           dynamicSamplingContext,
           ...evt.sdkProcessingMetadata,
         };
+
+        logger.log('xxx', JSON.stringify(evt.sdkProcessingMetadata, null, 2));
       }
       return evt;
     });

packages/core/src/envelope.ts

@@ -20,6 +20,7 @@ import {
   createEventEnvelopeHeaders,
   dsnToString,
   getSdkMetadataForEnvelopeHeader,
+  logger,
 } from '@sentry/utils';
 import { createSpanEnvelopeItem } from '@sentry/utils';
 import { getDynamicSamplingContextFromSpan } from './tracing/dynamicSamplingContext';
@@ -86,6 +87,9 @@ export function createEventEnvelope(
 
   const envelopeHeaders = createEventEnvelopeHeaders(event, sdkInfo, tunnel, dsn);
 
+  logger.log(JSON.stringify(envelopeHeaders, null, 2));
+  logger.log(JSON.stringify(event.sdkProcessingMetadata, null, 2));
+
   // Prevent this data (which, if it exists, was used in earlier steps in the processing pipeline) from being sent to
   // sentry. (Note: Our use of this property comes and goes with whatever we might be debugging, whatever hacks we may
   // have temporarily added, etc. Even if we don't happen to be using it at some point in the future, let's not get rid

packages/core/src/tracing/idleSpan.ts

@@ -239,11 +239,11 @@ export function startIdleSpan(startSpanOptions: StartSpanOptions, options: Parti
   }
 
   function onIdleSpanEnded(endTimestamp: number): void {
-    _cleanupHooks.forEach(cleanup => cleanup());
-
     _finished = true;
     activities.clear();
 
+    _cleanupHooks.forEach(cleanup => cleanup());
+
     _setSpanForScope(scope, previousActiveSpan);
 
     const spanJSON = spanToJSON(span);