Downgrade presence of guests and unsupported 3pids to warnings

Since migration is supported, but the data won't be used

Author: reivilibre

crates/syn2mas/src/synapse_reader/checks.rs

@@ -48,21 +48,11 @@ pub enum CheckError {
     )]
     PasswordSchemeWrongPepper,
 
-    #[error(
-        "Synapse database contains {num_guests} guests which aren't supported by MAS. See https://github.com/element-hq/matrix-authentication-service/issues/1445"
-    )]
-    GuestsInDatabase { num_guests: i64 },
-
     #[error(
         "Guest support is enabled in the Synapse configuration. Guests aren't supported by MAS, but if you don't have any then you could disable the option. See https://github.com/element-hq/matrix-authentication-service/issues/1445"
     )]
     GuestsEnabled,
 
-    #[error(
-        "Synapse database contains {num_non_email_3pids} non-email 3PIDs (probably phone numbers), which are not supported by MAS."
-    )]
-    NonEmailThreepidsInDatabase { num_non_email_3pids: i64 },
-
     #[error(
         "Synapse config has `enable_3pid_changes` explicitly enabled, which must be disabled or removed."
     )]
@@ -125,6 +115,16 @@ pub enum CheckWarning {
         "Synapse config has a registration CAPTCHA enabled, but no CAPTCHA has been configured in MAS. You may wish to manually configure this."
     )]
     ShouldPortRegistrationCaptcha,
+
+    #[error(
+        "Synapse database contains {num_guests} guests which will be migrated are not supported by MAS. See https://github.com/element-hq/matrix-authentication-service/issues/1445"
+    )]
+    GuestsInDatabase { num_guests: i64 },
+
+    #[error(
+        "Synapse database contains {num_non_email_3pids} non-email 3PIDs (probably phone numbers), which will be migrated but are not supported by MAS."
+    )]
+    NonEmailThreepidsInDatabase { num_non_email_3pids: i64 },
 }
 
 /// Check that the Synapse configuration is sane for migration.
@@ -260,21 +260,21 @@ pub async fn synapse_database_check(
     }
 
     let mut errors = Vec::new();
-    let warnings = Vec::new();
+    let mut warnings = Vec::new();
 
     let num_guests: i64 = query_scalar("SELECT COUNT(1) FROM users WHERE is_guest <> 0")
         .fetch_one(&mut *synapse_connection)
         .await?;
     if num_guests > 0 {
-        errors.push(CheckError::GuestsInDatabase { num_guests });
+        warnings.push(CheckWarning::GuestsInDatabase { num_guests });
     }
 
     let num_non_email_3pids: i64 =
         query_scalar("SELECT COUNT(1) FROM user_threepids WHERE medium <> 'email'")
             .fetch_one(&mut *synapse_connection)
             .await?;
     if num_non_email_3pids > 0 {
-        errors.push(CheckError::NonEmailThreepidsInDatabase {
+        warnings.push(CheckWarning::NonEmailThreepidsInDatabase {
             num_non_email_3pids,
         });
     }