From 77fd95ac4096faf1a641e0b7ec03541c3504bd5b Mon Sep 17 00:00:00 2001 From: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com> Date: Mon, 6 May 2024 12:39:36 -0400 Subject: [PATCH] fix(options): Remove "tunnel" from SDK options (#3787) * feat(ReactNativeClientOptions): Remove "tunnel" option Remove the "tunnel" option from the ReactNativeClientOptions type due to lack of support in React Native. * fix build * add changelog * fix link * fix lint --------- Co-authored-by: AliReza <38973917+AliReza99@users.noreply.github.com> --- CHANGELOG.md | 1 + src/js/client.ts | 2 +- src/js/options.ts | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8465b99141..5f39ec0a3f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,7 @@ ### Fixes +- Remove `tunnel` from SDK Options ([#3787](https://github.com/getsentry/sentry-react-native/pull/3787)) - Fix Apple non UIKit builds ([#3784](https://github.com/getsentry/sentry-react-native/pull/3784)) ### Dependencies diff --git a/src/js/client.ts b/src/js/client.ts index 9c62b5ea36..5512081e49 100644 --- a/src/js/client.ts +++ b/src/js/client.ts @@ -101,7 +101,7 @@ export class ReactNativeClient extends BaseClient { const envelope = createUserFeedbackEnvelope(feedback, { metadata: this._options._metadata, dsn: this.getDsn(), - tunnel: this._options.tunnel, + tunnel: undefined, }); this._sendEnvelope(envelope); } diff --git a/src/js/options.ts b/src/js/options.ts index 69a0fb5b2f..bf44620cd7 100644 --- a/src/js/options.ts +++ b/src/js/options.ts @@ -203,7 +203,9 @@ export interface ReactNativeTransportOptions extends BrowserTransportOptions { export interface ReactNativeOptions extends Options, BaseReactNativeOptions {} -export interface ReactNativeClientOptions extends ClientOptions, BaseReactNativeOptions {} +export interface ReactNativeClientOptions + extends Omit, 'tunnel'>, + BaseReactNativeOptions {} export interface ReactNativeWrapperOptions { /** Props for the root React profiler */