Merge 29f9cd1 into 1ec1b5d

getsentry · Jun 26, 2023 · 05075c5 · 05075c5
2 parents 1ec1b5d + 29f9cd1
commit 05075c5
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 8 deletions.
diff --git a/...-starter-jakarta/src/main/java/io/sentry/spring/boot/jakarta/SentryAutoConfiguration.java b/...-starter-jakarta/src/main/java/io/sentry/spring/boot/jakarta/SentryAutoConfiguration.java
@@ -119,9 +119,6 @@ static class HubConfiguration {
           BuildConfig.SENTRY_SPRING_BOOT_JAKARTA_SDK_NAME + "/" + BuildConfig.VERSION_NAME);
       options.setSdkVersion(createSdkVersion(options));
       addPackageAndIntegrationInfo();
-      if (options.getTracesSampleRate() == null && options.getEnableTracing() == null) {
-        options.setTracesSampleRate(0.0);
-      }
       // Spring Boot sets ignored exceptions in runtime using reflection - where the generic
       // information is lost
       // its technically possible to set non-throwable class to `ignoredExceptionsForType` set

diff --git a/...rter-jakarta/src/test/kotlin/io/sentry/spring/boot/jakarta/SentryAutoConfigurationTest.kt b/...rter-jakarta/src/test/kotlin/io/sentry/spring/boot/jakarta/SentryAutoConfigurationTest.kt
@@ -225,7 +225,7 @@ class SentryAutoConfigurationTest {
             "sentry.dsn=http://key@localhost/proj"
         ).run {
             val options = it.getBean(SentryProperties::class.java)
-            assertThat(options.tracesSampleRate).isNotNull().isEqualTo(0.0)
+            assertThat(options.tracesSampleRate).isNull()
         }
     }
 

diff --git a/sentry-spring-boot-starter/src/main/java/io/sentry/spring/boot/SentryAutoConfiguration.java b/sentry-spring-boot-starter/src/main/java/io/sentry/spring/boot/SentryAutoConfiguration.java
@@ -119,9 +119,6 @@ static class HubConfiguration {
           BuildConfig.SENTRY_SPRING_BOOT_SDK_NAME + "/" + BuildConfig.VERSION_NAME);
       options.setSdkVersion(createSdkVersion(options));
       addPackageAndIntegrationInfo();
-      if (options.getTracesSampleRate() == null && options.getEnableTracing() == null) {
-        options.setTracesSampleRate(0.0);
-      }
       // Spring Boot sets ignored exceptions in runtime using reflection - where the generic
       // information is lost
       // its technically possible to set non-throwable class to `ignoredExceptionsForType` set

diff --git a/...-spring-boot-starter/src/test/kotlin/io/sentry/spring/boot/SentryAutoConfigurationTest.kt b/...-spring-boot-starter/src/test/kotlin/io/sentry/spring/boot/SentryAutoConfigurationTest.kt
@@ -225,7 +225,7 @@ class SentryAutoConfigurationTest {
             "sentry.dsn=http://key@localhost/proj"
         ).run {
             val options = it.getBean(SentryProperties::class.java)
-            assertThat(options.tracesSampleRate).isNotNull().isEqualTo(0.0)
+            assertThat(options.tracesSampleRate).isNull()
         }
     }