Merge pull request 'only send mails to proper addresses' (#197) from staging into main

Reviewed-on: #197

src/model/mail.rs

@@ -8,7 +8,7 @@ use sqlx::SqlitePool;
 
 use crate::tera::admin::mail::MailToSend;
 
-use super::{family::Family, role::Role, user::User};
+use super::{family::Family, log::Log, role::Role, user::User};
 
 pub struct Mail {}
 
@@ -32,7 +32,16 @@ impl Mail {
         for rec in role.mails_from_role(db).await {
             let splitted = rec.split(',');
             for single_rec in splitted {
-                email = email.bcc(single_rec.parse().unwrap());
+                match single_rec.parse() {
+                    Ok(new_bcc_mail) => email = email.bcc(new_bcc_mail),
+                    Err(_) => {
+                        Log::create(
+                            db,
+                            format!("Mail not sent to {rec}, because it could not be parsed"),
+                        )
+                        .await;
+                    }
+                }
             }
         }