Skip to content

Commit f7a60fa

Browse files
committed
feat(core): Add getSdkMetadata to Client
1 parent b674c26 commit f7a60fa

File tree

4 files changed

+17
-3
lines changed

4 files changed

+17
-3
lines changed

packages/core/src/baseclient.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,15 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
219219
return this._options;
220220
}
221221

222+
/**
223+
* @see SdkMetadata in @sentry/types
224+
*
225+
* @return The metadata of the SDK
226+
*/
227+
public getSdkMetadata(): O['_metadata'] {
228+
return this._options._metadata;
229+
}
230+
222231
/**
223232
* @inheritDoc
224233
*/

packages/replay/src/util/getReplayEvent.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ export async function getReplayEvent({
2525
preparedEvent.platform = preparedEvent.platform || 'javascript';
2626

2727
// extract the SDK name because `client._prepareEvent` doesn't add it to the event
28-
const metadata = client.getOptions() && client.getOptions()._metadata;
28+
const metadata = client.getSdkMetadata && client.getSdkMetadata();
2929
const { name } = (metadata && metadata.sdk) || {};
3030

3131
preparedEvent.sdk = {

packages/types/src/client.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,13 @@ export interface Client<O extends ClientOptions = ClientOptions> {
6969
/** Returns the current options. */
7070
getOptions(): O;
7171

72+
/**
73+
* @inheritdoc
74+
*
75+
* TODO (v8): Make this a required method.
76+
*/
77+
getSdkMetadata?(): O['_metadata'];
78+
7279
/**
7380
* Returns the transport that is used by the client.
7481
* Please note that the transport gets lazy initialized so it will only be there once the first event has been sent.

rollup/plugins/bundlePlugins.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,6 @@ export function makeTerserPlugin() {
107107
'_driver',
108108
'_initStorage',
109109
'_support',
110-
// TODO: Get rid of these once we use the SDK to send replay events
111-
'_metadata', // replay uses client.getOptions()._metadata
112110
],
113111
},
114112
},

0 commit comments

Comments
 (0)