Skip to content

Commit c6a48f2

Browse files
committed
feat(replay): Move sample rate tags into replay_event
Similar to #6658 but waiting on getsentry/replay-backend#226
1 parent d15b5ec commit c6a48f2

File tree

3 files changed

+17
-21
lines changed

3 files changed

+17
-21
lines changed

packages/replay/src/replay.ts

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -932,11 +932,13 @@ export class ReplayContainer implements ReplayContainerInterface {
932932
...(includeReplayStartTimestamp ? { replay_start_timestamp: initialTimestamp / 1000 } : {}),
933933
timestamp: currentTimestamp / 1000,
934934
error_ids: errorIds,
935+
error_sample_rate: this._options.errorSampleRate,
935936
trace_ids: traceIds,
936937
urls,
937938
replay_id: replayId,
938-
segment_id,
939939
replay_type: this.session?.sampled,
940+
segment_id,
941+
session_sample_rate: this._options.sessionSampleRate,
940942
};
941943

942944
const replayEvent = await getReplayEvent({ scope, client, event: baseEvent });
@@ -948,12 +950,6 @@ export class ReplayContainer implements ReplayContainerInterface {
948950
return;
949951
}
950952

951-
replayEvent.tags = {
952-
...replayEvent.tags,
953-
sessionSampleRate: this._options.sessionSampleRate,
954-
errorSampleRate: this._options.errorSampleRate,
955-
};
956-
957953
/*
958954
For reference, the fully built event looks something like this:
959955
{
@@ -971,6 +967,8 @@ export class ReplayContainer implements ReplayContainerInterface {
971967
"replay_id": "eventId",
972968
"segment_id": 3,
973969
"replay_type": "error",
970+
"error_sample_rate": 1,
971+
"session_sample_rate": 1,
974972
"platform": "javascript",
975973
"event_id": "generated-uuid",
976974
"environment": "production",
@@ -983,10 +981,7 @@ export class ReplayContainer implements ReplayContainerInterface {
983981
"version": "7.25.0"
984982
},
985983
"sdkProcessingMetadata": {},
986-
"tags": {
987-
"sessionSampleRate": 1,
988-
"errorSampleRate": 0,
989-
}
984+
"tags": {}
990985
}
991986
*/
992987

packages/replay/test/unit/index.test.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -789,11 +789,9 @@ describe('Replay', () => {
789789
replayEventPayload: expect.objectContaining({
790790
replay_start_timestamp: (BASE_TIMESTAMP - 10000) / 1000,
791791
urls: ['http://localhost/'], // this doesn't truly test if we are capturing the right URL as we don't change URLs, but good enough
792+
error_sample_rate: 0,
792793
replay_type: 'session',
793-
tags: expect.objectContaining({
794-
errorSampleRate: 0,
795-
sessionSampleRate: 1,
796-
}),
794+
session_sample_rate: 1,
797795
}),
798796
});
799797
});

packages/replay/test/unit/util/createReplayEnvelope.test.ts

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,9 @@ describe('createReplayEnvelope', () => {
2424
version: '7.25.0',
2525
},
2626
replay_type: 'error',
27-
tags: {
28-
sessionSampleRate: 1,
29-
errorSampleRate: 0,
30-
},
27+
error_sample_rate: 0,
28+
session_sample_rate: 1,
29+
tags: {},
3130
};
3231

3332
const payloadWithSequence = 'payload';
@@ -56,13 +55,15 @@ describe('createReplayEnvelope', () => {
5655
{
5756
environment: 'production',
5857
error_ids: ['errorId'],
58+
error_sample_rate: 0,
5959
event_id: REPLAY_ID,
6060
platform: 'javascript',
6161
replay_id: REPLAY_ID,
6262
replay_type: 'error',
6363
sdk: { integrations: ['BrowserTracing', 'Replay'], name: 'sentry.javascript.browser', version: '7.25.0' },
6464
segment_id: 3,
65-
tags: { errorSampleRate: 0, sessionSampleRate: 1 },
65+
session_sample_rate: 1,
66+
tags: {},
6667
timestamp: 1670837008.634,
6768
trace_ids: ['traceId'],
6869
type: 'replay_event',
@@ -90,13 +91,15 @@ describe('createReplayEnvelope', () => {
9091
{
9192
environment: 'production',
9293
error_ids: ['errorId'],
94+
error_sample_rate: 0,
9395
event_id: REPLAY_ID,
9496
platform: 'javascript',
9597
replay_id: REPLAY_ID,
9698
sdk: { integrations: ['BrowserTracing', 'Replay'], name: 'sentry.javascript.browser', version: '7.25.0' },
9799
segment_id: 3,
98100
replay_type: 'error',
99-
tags: { errorSampleRate: 0, sessionSampleRate: 1 },
101+
session_sample_rate: 1,
102+
tags: {},
100103
timestamp: 1670837008.634,
101104
trace_ids: ['traceId'],
102105
type: 'replay_event',

0 commit comments

Comments
 (0)