Skip to content

Commit 8998fde

Browse files
authored
fix(core): Fix replay client report data category (#6891)
1 parent 9bb41df commit 8998fde

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

packages/core/src/baseclient.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -504,10 +504,12 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
504504
);
505505
}
506506

507+
const dataCategory: DataCategory = eventType === 'replay_event' ? 'replay' : eventType;
508+
507509
return this._prepareEvent(event, hint, scope)
508510
.then(prepared => {
509511
if (prepared === null) {
510-
this.recordDroppedEvent('event_processor', eventType, event);
512+
this.recordDroppedEvent('event_processor', dataCategory, event);
511513
throw new SentryError('An event processor returned `null`, will not send event.', 'log');
512514
}
513515

@@ -521,7 +523,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
521523
})
522524
.then(processedEvent => {
523525
if (processedEvent === null) {
524-
this.recordDroppedEvent('before_send', event.type || 'error', event);
526+
this.recordDroppedEvent('before_send', dataCategory, event);
525527
throw new SentryError(`${beforeSendLabel} returned \`null\`, will not send event.`, 'log');
526528
}
527529

packages/replay/src/util/sendReplayRequest.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export async function sendReplayRequest({
5858

5959
if (!replayEvent) {
6060
// Taken from baseclient's `_processEvent` method, where this is handled for errors/transactions
61-
client.recordDroppedEvent('event_processor', 'replay_event', baseEvent);
61+
client.recordDroppedEvent('event_processor', 'replay', baseEvent);
6262
__DEBUG_BUILD__ && logger.log('An event processor returned `null`, will not send event.');
6363
return;
6464
}

packages/types/src/datacategory.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export type DataCategory =
1111
// Transaction type event
1212
| 'transaction'
1313
// Replay type event
14-
| 'replay_event'
14+
| 'replay'
1515
// Events with `event_type` csp, hpkp, expectct, expectstaple
1616
| 'security'
1717
// Attachment bytes stored (unused for rate limiting
@@ -21,7 +21,4 @@ export type DataCategory =
2121
// SDK internal event, like client_reports
2222
| 'internal'
2323
// Profile event type
24-
| 'profile'
25-
// Replay event types
26-
| 'replay_event'
27-
| 'replay_recording';
24+
| 'profile';

packages/utils/src/envelope.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,8 +190,8 @@ const ITEM_TYPE_TO_DATA_CATEGORY_MAP: Record<EnvelopeItemType, DataCategory> = {
190190
client_report: 'internal',
191191
user_report: 'default',
192192
profile: 'profile',
193-
replay_event: 'replay_event',
194-
replay_recording: 'replay_recording',
193+
replay_event: 'replay',
194+
replay_recording: 'replay',
195195
};
196196

197197
/**

0 commit comments

Comments
 (0)