Skip to content

Commit

Permalink
Revert "sync timeouts to andorid"
Browse files Browse the repository at this point in the history
This reverts commit e7b8cb2.
  • Loading branch information
denrase committed Apr 18, 2023
1 parent e7b8cb2 commit 42cc429
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -186,9 +186,6 @@ class SentryFlutterPlugin : FlutterPlugin, MethodCallHandler, ActivityAware {
event
}

args.getIfNotNull<Int>("connectionTimeoutMillis") { options.connectionTimeoutMillis = it }
args.getIfNotNull<Int>("readTimeoutMillis") { options.connectionTimeoutMillis = it }

// missing proxy
}
result.success("")
Expand Down
2 changes: 0 additions & 2 deletions flutter/lib/src/integrations/native_sdk_integration.dart
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,6 @@ class NativeSdkIntegration extends Integration<SentryFlutterOptions> {
'maxAttachmentSize': options.maxAttachmentSize,
'captureFailedRequests': options.captureFailedRequests,
'enableAppHangTracking': options.enableAppHangTracking,
'connectionTimeoutMillis': options.connectionTimeout.inMilliseconds,
'readTimeoutMillis': options.readTimeout.inMilliseconds,
});

options.sdk.addIntegration('nativeSdkIntegration');
Expand Down
6 changes: 0 additions & 6 deletions flutter/lib/src/sentry_flutter_options.dart
Original file line number Diff line number Diff line change
Expand Up @@ -199,12 +199,6 @@ class SentryFlutterOptions extends SentryOptions {
/// Only available on iOS and macOS.
bool enableAppHangTracking = true;

/// Connection timeout. This will only be synced to the Android native SDK.
Duration connectionTimeout = Duration(seconds: 5);

/// Read timeout. This will only be synced to the Android native SDK.
Duration readTimeout = Duration(seconds: 5);

/// By using this, you are disabling native [Breadcrumb] tracking and instead
/// you are just tracking [Breadcrumb]s which result from events available
/// in the current Flutter environment.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ void main() {
'maxAttachmentSize': 20 * 1024 * 1024,
'captureFailedRequests': true,
'enableAppHangTracking': true,
'connectionTimeoutMillis': 5000,
'readTimeoutMillis': 5000,
});
});

Expand Down Expand Up @@ -98,9 +96,7 @@ void main() {
..proguardUuid = fakeProguardUuid
..maxAttachmentSize = 10
..captureFailedRequests = false
..enableAppHangTracking = false
..connectionTimeout = Duration(milliseconds: 9001)
..readTimeout = Duration(milliseconds: 9002);
..enableAppHangTracking = false;

options.sdk.addIntegration('foo');
options.sdk.addPackage('bar', '1');
Expand Down Expand Up @@ -141,8 +137,6 @@ void main() {
'maxAttachmentSize': 10,
'captureFailedRequests': false,
'enableAppHangTracking': false,
'connectionTimeoutMillis': 9001,
'readTimeoutMillis': 9002,
});
});

Expand Down

0 comments on commit 42cc429

Please sign in to comment.