Do not register sentrySpringFilter in ServletContext for Spring Boot (

#3027)
getsentry · Nov 6, 2023 · 2fad834 · 2fad834
1 parent 3d8bd2b
commit 2fad834
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+### Fixes
+
+- Do not register `sentrySpringFilter` in ServletContext for Spring Boot ([#3027](https://github.com/getsentry/sentry-java/pull/3027))
+
 ## 6.33.0
 
 ### Features

diff --git a/...karta/src/main/java/io/sentry/spring/jakarta/SentrySpringServletContainerInitializer.java b/...karta/src/main/java/io/sentry/spring/jakarta/SentrySpringServletContainerInitializer.java
@@ -1,5 +1,7 @@
 package io.sentry.spring.jakarta;
 
+import static io.sentry.util.ClassLoaderUtils.classLoaderOrDefault;
+
 import com.jakewharton.nopen.annotation.Open;
 import jakarta.servlet.DispatcherType;
 import jakarta.servlet.FilterRegistration;
@@ -20,10 +22,17 @@ public class SentrySpringServletContainerInitializer implements ServletContainer
   @Override
   public void onStartup(final @Nullable Set<Class<?>> c, final @NotNull ServletContext ctx)
       throws ServletException {
-    final FilterRegistration.Dynamic dynamic =
-        ctx.addFilter("sentrySpringFilter", SentrySpringFilter.class);
-    if (dynamic != null) {
-      dynamic.addMappingForUrlPatterns(EnumSet.allOf(DispatcherType.class), false, "/*");
+    try {
+      Class.forName(
+          "org.springframework.boot.SpringApplication",
+          false,
+          classLoaderOrDefault(getClass().getClassLoader()));
+    } catch (ClassNotFoundException e) {
+      final FilterRegistration.Dynamic dynamic =
+          ctx.addFilter("sentrySpringFilter", SentrySpringFilter.class);
+      if (dynamic != null) {
+        dynamic.addMappingForUrlPatterns(EnumSet.allOf(DispatcherType.class), false, "/*");
+      }
     }
   }
 }