Skip to content

Commit ca561f3

Browse files
committed
use new field for transaction sampling data
1 parent e52faa4 commit ca561f3

File tree

4 files changed

+5
-34
lines changed

4 files changed

+5
-34
lines changed

packages/core/src/request.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,8 @@ export function eventToSentryRequest(event: Event, api: APIDetails): SentryReque
5555
const eventType = event.type || 'event';
5656
const useEnvelope = eventType === 'transaction' || !!api.tunnel;
5757

58-
const { transactionSampling, ...metadata } = event.debug_meta || {};
58+
const { transactionSampling } = event.sdkProcessingMetadata || {};
5959
const { method: samplingMethod, rate: sampleRate } = transactionSampling || {};
60-
if (Object.keys(metadata).length === 0) {
61-
delete event.debug_meta;
62-
} else {
63-
event.debug_meta = metadata;
64-
}
6560

6661
// prevent this data from being sent to sentry
6762
delete event.sdkProcessingMetadata;

packages/core/test/lib/request.test.ts

Lines changed: 3 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { DebugMeta, Event, SentryRequest } from '@sentry/types';
1+
import { Event, SentryRequest } from '@sentry/types';
22

33
import { initAPIDetails } from '../../src/api';
44
import { eventToSentryRequest, sessionToSentryRequest } from '../../src/request';
@@ -40,11 +40,12 @@ describe('eventToSentryRequest', () => {
4040
transaction: '/dogs/are/great/',
4141
type: 'transaction',
4242
user: { id: '1121', username: 'CharlieDog', ip_address: '11.21.20.12' },
43+
sdkProcessingMetadata: {},
4344
};
4445
});
4546

4647
it('adds transaction sampling information to item header', () => {
47-
event.debug_meta = { transactionSampling: { method: 'client_rate', rate: 0.1121 } };
48+
event.sdkProcessingMetadata = { transactionSampling: { method: 'client_rate', rate: 0.1121 } };
4849

4950
const result = eventToSentryRequest(event, api);
5051
const envelope = parseEnvelopeRequest(result);
@@ -56,30 +57,6 @@ describe('eventToSentryRequest', () => {
5657
);
5758
});
5859

59-
it('removes transaction sampling information (and only that) from debug_meta', () => {
60-
event.debug_meta = {
61-
transactionSampling: { method: 'client_sampler', rate: 0.1121 },
62-
dog: 'Charlie',
63-
} as DebugMeta;
64-
65-
const result = eventToSentryRequest(event, api);
66-
const envelope = parseEnvelopeRequest(result);
67-
68-
expect('transactionSampling' in envelope.event.debug_meta).toBe(false);
69-
expect('dog' in envelope.event.debug_meta).toBe(true);
70-
});
71-
72-
it('removes debug_meta entirely if it ends up empty', () => {
73-
event.debug_meta = {
74-
transactionSampling: { method: 'client_rate', rate: 0.1121 },
75-
} as DebugMeta;
76-
77-
const result = eventToSentryRequest(event, api);
78-
const envelope = parseEnvelopeRequest(result);
79-
80-
expect('debug_meta' in envelope.event).toBe(false);
81-
});
82-
8360
it('adds sdk info to envelope header', () => {
8461
const result = eventToSentryRequest(event, api);
8562
const envelope = parseEnvelopeRequest(result);

packages/tracing/src/transaction.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ export class Transaction extends SpanClass implements TransactionInterface {
132132
timestamp: this.endTimestamp,
133133
transaction: this.name,
134134
type: 'transaction',
135-
debug_meta: this.metadata,
135+
sdkProcessingMetadata: this.metadata,
136136
};
137137

138138
const hasMeasurements = Object.keys(this._measurements).length > 0;

packages/types/src/debugMeta.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
**/
44
export interface DebugMeta {
55
images?: Array<DebugImage>;
6-
transactionSampling?: { rate?: number; method?: string };
76
}
87

98
/**

0 commit comments

Comments
 (0)