Remove unnecessary unwraps

src/email.rs

@@ -30,11 +30,10 @@ async fn send_email(
     config: &AybConfigEmail,
     e2e_testing_on: bool,
 ) -> Result<(), AybError> {
-    // TODO(marcua): Any way to be more careful about these unwraps?
     let email = Message::builder()
-        .from(config.from.parse().unwrap())
-        .reply_to(config.reply_to.parse().unwrap())
-        .to(to.parse().unwrap())
+        .from(config.from.parse()?)
+        .reply_to(config.reply_to.parse()?)
+        .to(to.parse()?)
         .subject(subject)
         .header(ContentType::TEXT_PLAIN)
         .body(body)

src/error.rs

@@ -1,6 +1,7 @@
 use actix_web;
 use derive_more::{Display, Error};
 use fernet;
+use lettre;
 use quoted_printable;
 use reqwest;
 use rusqlite;
@@ -28,6 +29,14 @@ impl From<fernet::DecryptionError> for AybError {
     }
 }
 
+impl From<lettre::address::AddressError> for AybError {
+    fn from(cause: lettre::address::AddressError) -> Self {
+        AybError {
+            message: format!("Invalid email address: {}", cause),
+        }
+    }
+}
+
 impl From<quoted_printable::QuotedPrintableError> for AybError {
     fn from(cause: quoted_printable::QuotedPrintableError) -> Self {
         AybError {