Skip to content

Commit 0b37db6

Browse files
adinauerclaude
andcommitted
ref(core): Remove legacy Logs warning inputs
Remove obsolete external and Android manifest parsing retained only for migration warnings that will not ship in v9. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent cd831b8 commit 0b37db6

5 files changed

Lines changed: 0 additions & 21 deletions

File tree

‎sentry-android-core/src/main/java/io/sentry/android/core/ManifestMetadataReader.java‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -192,8 +192,6 @@ final class ManifestMetadataReader {
192192

193193
static final String IN_APP_EXCLUDES = "io.sentry.in-app-excludes";
194194

195-
static final String ENABLE_LOGS = "io.sentry.logs.enabled";
196-
197195
static final String ENABLE_TIMBER_LOGS = "io.sentry.timber.logs.enabled";
198196

199197
static final String ENABLE_LOGCAT_LOGS = "io.sentry.logcat.logs.enabled";

‎sentry/api/sentry.api‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -590,7 +590,6 @@ public final class io/sentry/ExternalOptions {
590590
public fun isEnableBackpressureHandling ()Ljava/lang/Boolean;
591591
public fun isEnableCacheTracing ()Ljava/lang/Boolean;
592592
public fun isEnableDatabaseTransactionTracing ()Ljava/lang/Boolean;
593-
public fun isEnableLogs ()Ljava/lang/Boolean;
594593
public fun isEnableMetrics ()Ljava/lang/Boolean;
595594
public fun isEnablePrettySerializationOutput ()Ljava/lang/Boolean;
596595
public fun isEnableQueueTracing ()Ljava/lang/Boolean;
@@ -611,7 +610,6 @@ public final class io/sentry/ExternalOptions {
611610
public fun setEnableCacheTracing (Ljava/lang/Boolean;)V
612611
public fun setEnableDatabaseTransactionTracing (Ljava/lang/Boolean;)V
613612
public fun setEnableDeduplication (Ljava/lang/Boolean;)V
614-
public fun setEnableLogs (Ljava/lang/Boolean;)V
615613
public fun setEnableMetrics (Ljava/lang/Boolean;)V
616614
public fun setEnablePrettySerializationOutput (Ljava/lang/Boolean;)V
617615
public fun setEnableQueueTracing (Ljava/lang/Boolean;)V

‎sentry/src/main/java/io/sentry/ExternalOptions.java‎

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ public final class ExternalOptions {
4646
private @Nullable Boolean enabled;
4747
private @Nullable Boolean enablePrettySerializationOutput;
4848
private @Nullable Boolean enableSpotlight;
49-
private @Nullable Boolean enableLogs;
5049
private @Nullable Boolean enableMetrics;
5150
private @Nullable String spotlightConnectionUrl;
5251

@@ -178,8 +177,6 @@ public final class ExternalOptions {
178177
options.setCaptureOpenTelemetryEvents(
179178
propertiesProvider.getBooleanProperty("capture-open-telemetry-events"));
180179

181-
options.setEnableLogs(propertiesProvider.getBooleanProperty("logs.enabled"));
182-
183180
options.setEnableMetrics(propertiesProvider.getBooleanProperty("metrics.enabled"));
184181

185182
for (final String ignoredExceptionType :
@@ -717,14 +714,6 @@ public void setCaptureOpenTelemetryEvents(final @Nullable Boolean captureOpenTel
717714
return captureOpenTelemetryEvents;
718715
}
719716

720-
public void setEnableLogs(final @Nullable Boolean enableLogs) {
721-
this.enableLogs = enableLogs;
722-
}
723-
724-
public @Nullable Boolean isEnableLogs() {
725-
return enableLogs;
726-
}
727-
728717
public void setEnableMetrics(final @Nullable Boolean enableMetrics) {
729718
this.enableMetrics = enableMetrics;
730719
}

‎sentry/src/test/java/io/sentry/ExternalOptionsTest.kt‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -530,11 +530,6 @@ class ExternalOptionsTest {
530530
}
531531
}
532532

533-
@Test
534-
fun `creates options with enableLogs set to true`() {
535-
withPropertiesFile("logs.enabled=true") { options -> assertTrue(options.isEnableLogs == true) }
536-
}
537-
538533
@Test
539534
fun `creates options with enableMetrics set to true`() {
540535
withPropertiesFile("metrics.enabled=true") { options ->

‎sentry/src/test/java/io/sentry/SentryOptionsTest.kt‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -579,7 +579,6 @@ class SentryOptionsTest {
579579
externalOptions.isEnableSpotlight = true
580580
externalOptions.spotlightConnectionUrl = "http://local.sentry.io:1234"
581581
externalOptions.isGlobalHubMode = true
582-
externalOptions.isEnableLogs = true
583582
externalOptions.isEnableMetrics = false
584583
externalOptions.profileSessionSampleRate = 0.8
585584
externalOptions.profilingTracesDirPath = "/profiling-traces"

0 commit comments

Comments
 (0)