Enable ThreadLocalAccessor for Spring Boot 3 WebFlux by default

Author: adinauer

sentry-spring-boot-jakarta/src/main/java/io/sentry/spring/boot/jakarta/SentryWebfluxAutoConfiguration.java

@@ -91,8 +91,7 @@ public SentryLegacyFilterConfigurationCondition() {
 
     @ConditionalOnProperty(
         name = "sentry.reactive.thread-local-accessor-enabled",
-        havingValue = "false",
-        matchIfMissing = true)
+        havingValue = "false")
     @SuppressWarnings("UnusedNestedClass")
     private static class SentryDisableThreadLocalAccessorCondition {}
 
@@ -109,7 +108,8 @@ public SentryThreadLocalAccessorCondition() {
 
     @ConditionalOnProperty(
         name = "sentry.reactive.thread-local-accessor-enabled",
-        havingValue = "true")
+        havingValue = "true",
+        matchIfMissing = true)
     @SuppressWarnings("UnusedNestedClass")
     private static class SentryEnableThreadLocalAccessorCondition {}
 

sentry-spring-boot-jakarta/src/test/kotlin/io/sentry/spring/boot/jakarta/SentryWebfluxAutoConfigurationTest.kt

@@ -21,8 +21,8 @@ class SentryWebfluxAutoConfigurationTest {
     fun `configures sentryWebFilter`() {
         contextRunner.withPropertyValues("sentry.dsn=http://key@localhost/proj")
             .run {
-                assertThat(it).hasSingleBean(SentryWebFilter::class.java)
-                assertThat(it).doesNotHaveBean(SentryWebFilterWithThreadLocalAccessor::class.java)
+                assertThat(it).hasSingleBean(SentryWebFilterWithThreadLocalAccessor::class.java)
+                assertThat(it).doesNotHaveBean(SentryWebFilter::class.java)
             }
     }
 
@@ -63,6 +63,7 @@ class SentryWebfluxAutoConfigurationTest {
             )
             .run {
                 assertThat(it).hasSingleBean(SentryWebFilterWithThreadLocalAccessor::class.java)
+                assertThat(it).doesNotHaveBean(SentryWebFilter::class.java)
             }
     }
 
@@ -75,6 +76,7 @@ class SentryWebfluxAutoConfigurationTest {
             )
             .run {
                 assertThat(it).doesNotHaveBean(SentryWebFilterWithThreadLocalAccessor::class.java)
+                assertThat(it).hasSingleBean(SentryWebFilter::class.java)
             }
     }