Handle AS users with invalid localparts edge case

Author: sandhose

crates/syn2mas/src/migration.rs

@@ -170,6 +170,14 @@ async fn migrate_users(
 
     while let Some(user_res) = users_stream.next().await {
         let user = user_res.into_synapse("reading user")?;
+
+        // Handling an edge case: some AS users may have invalid localparts containing
+        // extra `:` characters. These users are ignored and a warning is logged.
+        if user.appservice_id.is_some() && user.name.0.chars().filter(|c| *c == ':').count() > 1 {
+            tracing::warn!("AS user {} has invalid localpart, ignoring!", user.name.0);
+            continue;
+        }
+
         let (mas_user, mas_password_opt) = transform_user(&user, &state.server_name, rng)?;
 
         let mut flags = UserFlags::empty();