Skip to content

Commit

Permalink
fix: disable sentry on iOS
Browse files Browse the repository at this point in the history
getsentry/sentry-cocoa#1892 is not fixed, or
the fix was not enough for us to get sentry back working. We still
get crash reports where the source of the crash is reported as
Sentry itself.
  • Loading branch information
jaroslavas committed Feb 28, 2024
1 parent 69e203e commit 6356266
Showing 1 changed file with 13 additions and 11 deletions.
24 changes: 13 additions & 11 deletions apps/ios/ProtonVPN/AppDelegate.swift
Original file line number Diff line number Diff line change
Expand Up @@ -104,17 +104,19 @@ extension AppDelegate: UIApplicationDelegate {
// Protocol check is placed here for parity with MacOS
adjustGlobalProtocolIfNecessary()

if FeatureFlagsRepository.shared.isEnabled(VPNFeatureFlagType.sentry) {
SentryHelper.setupSentry(
dsn: ObfuscatedConstants.sentryDsniOS,
isEnabled: { [weak self] in
self?.container.makeTelemetrySettings().telemetryCrashReports ?? false
},
getUserId: { [weak self] in
self?.container.makeAuthKeychainHandle().userId
}
)
}
// Sentry turned off, because https://github.com/getsentry/sentry-cocoa/issues/1892
// is still not fixed.
// if FeatureFlagsRepository.shared.isEnabled(VPNFeatureFlagType.sentry) {
// SentryHelper.setupSentry(
// dsn: ObfuscatedConstants.sentryDsniOS,
// isEnabled: { [weak self] in
// self?.container.makeTelemetrySettings().telemetryCrashReports ?? false
// },
// getUserId: { [weak self] in
// self?.container.makeAuthKeychainHandle().userId
// }
// )
// }

AnnouncementButtonViewModel.shared = container.makeAnnouncementButtonViewModel()

Expand Down

0 comments on commit 6356266

Please sign in to comment.