fix: dont break on user not still being in channel and force disconnecting

Cargo.lock

@@ -6905,7 +6905,6 @@ dependencies = [
  "revolt_rocket_okapi",
  "rocket",
  "schemars 0.8.22",
- "sentry",
  "serde",
  "serde_json",
  "utoipa",

crates/core/database/src/events/client.rs

@@ -295,9 +295,7 @@ pub enum EventV1 {
     },
     UserMoveVoiceChannel {
         node: String,
-        from: String,
-        to: String,
-        token: String,
+        token: String
     }
 }
 

crates/core/database/src/models/admin_migrations/ops/mongodb/scripts.rs

@@ -25,7 +25,7 @@ struct MigrationInfo {
     revision: i32,
 }
 
-pub const LATEST_REVISION: i32 = 49; // MUST BE +1 to last migration
+pub const LATEST_REVISION: i32 = 48; // MUST BE +1 to last migration
 
 pub async fn migrate_database(db: &MongoDb) {
     let migrations = db.col::<Document>("migrations");
@@ -1246,8 +1246,8 @@ pub async fn run_migrations(db: &MongoDb, revision: i32) -> i32 {
             .expect("Failed to update voice channels");
     };
 
-        if revision <= 48 {
-        info!("Running migration [revision 48 / 22-10-2025]: Add Video + Listen to default permissions");
+        if revision <= 47 {
+        info!("Running migration [revision 47 / 29-04-2025]: Add Video to default permissions");
 
         db.col::<Document>("servers")
             .update_many(
@@ -1255,7 +1255,7 @@ pub async fn run_migrations(db: &MongoDb, revision: i32) -> i32 {
                 doc! {
                     "$bit": {
                         "default_permissions": {
-                            "or": (ChannelPermission::Video + ChannelPermission::Speak + ChannelPermission::Listen) as i64
+                            "or": ChannelPermission::Video as i64
                         },
                     }
                 }

crates/core/database/src/voice/mod.rs

@@ -16,7 +16,7 @@ mod voice_client;
 pub use voice_client::VoiceClient;
 
 async fn get_connection() -> Result<Conn> {
-    _get_connection().await.map_err(|_| create_error!(InternalError))
+    _get_connection().await.to_internal_error()
 }
 
 pub async fn raise_if_in_voice(user: &User, channel_id: &str) -> Result<()> {

crates/core/result/Cargo.toml

@@ -15,9 +15,8 @@ utoipa = ["dep:utoipa"]
 rocket = ["dep:rocket", "dep:serde_json"]
 axum = ["dep:axum", "dep:serde_json"]
 okapi = ["dep:revolt_rocket_okapi", "dep:revolt_okapi", "schemas"]
-sentry = ["dep:sentry"]
 
-default = ["serde", "sentry"]
+default = ["serde"]
 
 [dependencies]
 # Serialisation
@@ -37,5 +36,3 @@ revolt_okapi = { version = "0.9.1", optional = true }
 log = "0.4"
 # Axum
 axum = { version = "0.7.5", optional = true }
-
-sentry = { version = "0.31.5", optional = true }

crates/core/result/src/lib.rs

@@ -226,7 +226,7 @@ pub trait ToRevoltError<T> {
     fn to_internal_error(self) -> Result<T, Error>;
 }
 
-impl<T, E: std::fmt::Debug + std::error::Error> ToRevoltError<T> for Result<T, E> {
+impl<T, E: std::fmt::Debug> ToRevoltError<T> for Result<T, E> {
     #[track_caller]
     fn to_internal_error(self) -> Result<T, Error> {
         let loc = Location::caller();
@@ -234,8 +234,6 @@ impl<T, E: std::fmt::Debug + std::error::Error> ToRevoltError<T> for Result<T, E
         self
             .map_err(|e| {
                 log::error!("{e:?}");
-                #[cfg(feature = "sentry")]
-                sentry::capture_error(&e);
 
                 Error {
                     error_type: ErrorType::InternalError,

crates/daemons/voice-ingress/src/api.rs

@@ -199,8 +199,8 @@ pub async fn ingress(
                 };
             }
         }
-        // Audio/video track was started/stopped/unmuted/muted
-        "track_published" | "track_unpublished" | "track_unmuted" | "track_muted" => {
+        // Audio/video track was started/stopped
+        "track_published" | "track_unpublished" => {
             let channel_id = channel_id.to_internal_error()?;
             let user_id = user_id.to_internal_error()?;
             let track = event.track.as_ref().to_internal_error()?;
@@ -255,7 +255,7 @@ pub async fn ingress(
                 channel_id,
                 channel.server(),
                 user_id,
-                event.event == "track_published" || event.event == "track_unmuted", // to avoid duplicating this entire case twice
+                event.event == "track_published", // to avoid duplicating this entire case twice
                 track.source,
             )
             .await?;

crates/delta/src/routes/channels/voice_join.rs

@@ -103,7 +103,7 @@ pub async fn call(
     log::debug!("Created room {}", room.name);
 
     if let Some(recipients) = recipients {
-        if room.num_participants == 0 && !recipients.is_empty() {
+        if room.num_participants == 0 {
             set_call_notification_recipients(channel.id(), &user.id, &recipients).await?;
         }
     }

crates/delta/src/routes/servers/ban_create.rs

@@ -59,7 +59,7 @@ pub async fn ban(
 
         // If the member is in a voice channel while banned kick them from the voice channel
         if let Some(channel_id) = get_user_voice_channel_in_server(&user.id, &server.id).await? {
-            if let Some(node) = get_channel_node(&channel_id).await? {
+            if let Some(node) = get_channel_node(&channel_id).await?.unwrap() {
                 let _ = voice_client.remove_user(&node, &user.id, &channel_id).await;
             }
 

crates/delta/src/routes/servers/member_edit.rs

@@ -217,11 +217,9 @@ pub async fn edit(
 
             EventV1::UserMoveVoiceChannel {
                 node: new_node,
-                from: channel,
-                to: new_voice_channel.id().to_string(),
                 token,
             }
-            .private(target_user.id.clone())
+            .p_user(target_user.id.clone(), db)
             .await;
         };
     } else if can_publish.is_some() || can_receive.is_some() || remove_contains_voice {