Upgrade Sentry to 0.42.0 (element-hq#4844)

sandhose · web-flow · commit a995d92eb0af · 2025-07-30T14:33:51.000+02:00
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -572,18 +572,18 @@ features = [
 
 # Sentry error tracking
 [workspace.dependencies.sentry]
-version = "0.37.0"
+version = "0.42.0"
 default-features = false
 features = ["backtrace", "contexts", "panic", "tower", "reqwest"]
 
 # Sentry tower layer
 [workspace.dependencies.sentry-tower]
-version = "0.37.0"
+version = "0.42.0"
 features = ["http", "axum-matched-path"]
 
 # Sentry tracing integration
 [workspace.dependencies.sentry-tracing]
-version = "0.37.0"
+version = "0.42.0"
 
 # Serialization and deserialization
 [workspace.dependencies.serde]
@@ -645,7 +645,7 @@ features = [
 version = "2.0.12"
 
 [workspace.dependencies.thiserror-ext]
-version = "0.2.1"
+version = "0.3.0"
 
 # Async runtime
 [workspace.dependencies.tokio]
diff --git a/crates/cli/src/main.rs b/crates/cli/src/main.rs
@@ -128,8 +128,6 @@ async fn try_main() -> anyhow::Result<ExitCode> {
             release: Some(VERSION.into()),
             sample_rate: telemetry_config.sentry.sample_rate.unwrap_or(1.0),
             traces_sample_rate: telemetry_config.sentry.traces_sample_rate.unwrap_or(0.0),
-            auto_session_tracking: true,
-            session_mode: sentry::SessionMode::Request,
             ..Default::default()
         },
     ));
diff --git a/crates/cli/src/server.rs b/crates/cli/src/server.rs
@@ -333,7 +333,7 @@ pub fn build_router(
         // which is the other way around compared to `tower::ServiceBuilder`.
         // So even if the Sentry docs has an example that does
         // 'NewSentryHttpLayer then SentryHttpLayer', we must do the opposite.
-        .layer(SentryHttpLayer::with_transaction())
+        .layer(SentryHttpLayer::new().enable_transaction())
         .layer(NewSentryLayer::new_from_top())
         .with_state(state)
 }
diff --git a/deny.toml b/deny.toml
@@ -60,7 +60,6 @@ multiple-versions = "deny"
 skip = [
     { name = "regex-syntax", version = "0.6.29" },   # tracing-subscriber[env-filter] -> matchers depends on the old version
     { name = "regex-automata", version = "0.1.10" }, # ^
-    { name = "itertools", version = "0.12.1" },      # otel -> prost depends on this old version
     { name = "itertools", version = "0.13.0" },      # zxcvbn depends on this old version
     { name = "indexmap", version = "1.9.3" },        # schemars depends on this old version
     { name = "hashbrown", version = "0.12.3" },      # schemars -> indexmap depends on this old version

Original file line number	Diff line number	Diff line change
`@@ -333,7 +333,7 @@ pub fn build_router(`
`333`	`333`	// which is the other way around compared to `tower::ServiceBuilder`.
`334`	`334`	`// So even if the Sentry docs has an example that does`
`335`	`335`	`// 'NewSentryHttpLayer then SentryHttpLayer', we must do the opposite.`
`336`		`- .layer(SentryHttpLayer::with_transaction())`
	`336`	`+ .layer(SentryHttpLayer::new().enable_transaction())`
`337`	`337`	`.layer(NewSentryLayer::new_from_top())`
`338`	`338`	`.with_state(state)`
`339`	`339`	`}`