OBPIH-6733 sentry performance fix (#4944)

openboxes · Nov 15, 2024 · 79a3149 · 79a3149
1 parent edfb53f
commit 79a3149
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 56 deletions.
diff --git a/build.gradle b/build.gradle
@@ -496,7 +496,11 @@ dependencies {
     implementation "io.sentry:sentry-servlet:${sentryVersion}"
     // Uses P6Spy to capture database queries.
     implementation "io.sentry:sentry-jdbc:${sentryVersion}"
-    implementation "io.sentry:sentry-spring-boot-starter:${sentryVersion}"
+    // We can't use "io.sentry:sentry-spring-boot-starter" because it causes a null pointer on transaction name in
+    // io.sentry.spring.boot.SentryAutoConfiguration$HubConfiguration$SentryWebMvcConfiguration.transactionNameProvider
+    // but only when deploying to an external Servlet. Once this is resolved we can switch to sentry-spring-boot-starter
+    // which will give us automatic Sentry releases and allow us to configure sentry in application.yml
+    implementation "io.sentry:sentry-spring:${sentryVersion}"
 
     // incorporate latest timezone changes
     implementation 'joda-time:joda-time:2.12.5'

diff --git a/src/main/groovy/org/pih/warehouse/monitoring/SentryServletContainerInitializer.groovy b/src/main/groovy/org/pih/warehouse/monitoring/SentryServletContainerInitializer.groovy
diff --git a/src/main/resources/META-INF/services/javax.servlet.ServletContainerInitializer b/src/main/resources/META-INF/services/javax.servlet.ServletContainerInitializer