Add delivery_email to database.

This ensures that we have one case-insensitive email to compare against and one case-sensitive to send emails to.

backend/src/lib.rs

@@ -390,16 +390,18 @@ pub(crate) mod tests {
     #[actix_web::test]
     async fn test_clean_verification_tokens() {
         let user_id = uuid::Uuid::new_v4();
+        let email = format!("{}@invalid", user_id.to_string());
         let user = User {
             id: user_id,
             name: user_id.to_string(),
-            email: format!("{}@invalid", user_id.to_string()),
+            email: email.clone(),
             login_key: String::new(),
             email_verified: false,
             secret: Vec::new(),
             secret_nonce: Vec::new(),
             secret_salt: Vec::new(),
             login_salt: Vec::new(),
+            delivery_email: Some(email),
         };
 
         let user2_id = uuid::Uuid::new_v4();
@@ -413,6 +415,7 @@ pub(crate) mod tests {
             secret_nonce: Vec::new(),
             secret_salt: Vec::new(),
             login_salt: Vec::new(),
+            delivery_email: None,
         };
 
         let user3_id = uuid::Uuid::new_v4();
@@ -426,6 +429,7 @@ pub(crate) mod tests {
             secret_nonce: Vec::new(),
             secret_salt: Vec::new(),
             login_salt: Vec::new(),
+            delivery_email: None,
         };
 
         let verify_id = uuid::Uuid::new_v4();

backend/src/routes/auth/register.rs

@@ -183,6 +183,7 @@ pub async fn register(
         secret_nonce: data.secret_nonce.clone(),
         login_salt: data.login_salt.clone(),
         secret_salt: data.secret_salt.clone(),
+        delivery_email: Some(data.email.clone()),
     };
 
     let mut conn = get_connection(&state)?;
@@ -226,7 +227,7 @@ pub async fn register(
             send_verification_mail(
                 user_insert.name,
                 verify,
-                mail,
+                data.email.clone(),
                 state.mailer.clone(),
                 state.frontend_url.clone(),
                 lang.into(),

backend/src/routes/auth/start_recovery.rs

@@ -137,10 +137,16 @@ pub async fn start_recovery(
 
     #[cfg(not(test))]
     std::thread::spawn(move || {
+        let email = if let Some(email) = user.delivery_email {
+            email
+        } else {
+            user.email
+        };
+
         send_email(
             user.name,
             token_id,
-            user.email,
+            email,
             state.mailer.clone(),
             state.frontend_url.clone(),
             lang.into(),

db_connector/migrations/2024-12-18-092514_users_add_delivery_email/down.sql

@@ -0,0 +1,2 @@
+-- This file should undo anything in `up.sql`
+ALTER TABLE "users" DROP COLUMN "delivery_email";

db_connector/migrations/2024-12-18-092514_users_add_delivery_email/up.sql

@@ -0,0 +1,2 @@
+-- Your SQL goes here
+ALTER TABLE "users" ADD COLUMN "delivery_email" VARCHAR;

db_connector/src/models/users.rs

@@ -13,4 +13,5 @@ pub struct User {
     pub secret_nonce: Vec<u8>,
     pub secret_salt: Vec<u8>,
     pub login_salt: Vec<u8>,
+    pub delivery_email: Option<String>,
 }

db_connector/src/schema.rs

@@ -58,6 +58,7 @@ diesel::table! {
         secret_salt -> Bytea,
         #[sql_name = "login-salt"]
         login_salt -> Bytea,
+        delivery_email -> Nullable<Varchar>,
     }
 }