Adjust mail transport error handling

Gotta keep the linter happy :)

Signed-off-by: Erin Allison <eallison@andrettikarting.com>

Author: erin-allison

internal-packages/emails/src/transports/aws-ses.ts

@@ -31,11 +31,15 @@ export class AwsSesMailTransport implements MailTransport {
         html: render(react),
       });
     }
-    catch (error: Error) {
-      console.error(
-        `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
-      );
-      throw new EmailError(error);
+    catch (error) {
+      if (error instanceof Error) {
+        console.error(
+          `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
+        );
+        throw new EmailError(error);
+      } else {
+        throw error;
+      }
     }
   }
 
@@ -49,11 +53,15 @@ export class AwsSesMailTransport implements MailTransport {
         text: text,
       });
     }
-    catch (error: Error) {
-      console.error(
-        `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
-      );
-      throw new EmailError(error);
+    catch (error) {
+      if (error instanceof Error) {
+        console.error(
+          `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
+        );
+        throw new EmailError(error);
+      } else {
+        throw error;
+      }
     }
   }
 }

internal-packages/emails/src/transports/smtp.ts

@@ -1,28 +1,28 @@
 import { render } from "@react-email/render";
+import nodemailer from "nodemailer";
 import { EmailError, MailMessage, MailTransport, PlainTextMailMessage } from "./index";
-import nodemailer from "nodemailer"
 
 export type SmtpMailTransportOptions = {
-  type: 'smtp',
+  type: "smtp";
   config: {
-    host?: string,
-    port?: number,
-    secure?: boolean,
+    host?: string;
+    port?: number;
+    secure?: boolean;
     auth?: {
-      user?: string,
-      pass?: string
-    }
-  }
-}
+      user?: string;
+      pass?: string;
+    };
+  };
+};
 
 export class SmtpMailTransport implements MailTransport {
   #client: nodemailer.Transporter;
 
   constructor(options: SmtpMailTransportOptions) {
-    this.#client = nodemailer.createTransport(options.config)
+    this.#client = nodemailer.createTransport(options.config);
   }
 
-  async send({to, from, replyTo, subject, react}: MailMessage): Promise<void> {
+  async send({ to, from, replyTo, subject, react }: MailMessage): Promise<void> {
     try {
       await this.#client.sendMail({
         from: from,
@@ -31,16 +31,19 @@ export class SmtpMailTransport implements MailTransport {
         subject,
         html: render(react),
       });
-    }
-    catch (error: Error) {
-      console.error(
-        `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
-      );
-      throw new EmailError(error);
+    } catch (error) {
+      if (error instanceof Error) {
+        console.error(
+          `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
+        );
+        throw new EmailError(error);
+      } else {
+        throw error;
+      }
     }
   }
 
-  async sendPlainText({to, from, replyTo, subject, text}: PlainTextMailMessage): Promise<void> {
+  async sendPlainText({ to, from, replyTo, subject, text }: PlainTextMailMessage): Promise<void> {
     try {
       await this.#client.sendMail({
         from: from,
@@ -49,12 +52,15 @@ export class SmtpMailTransport implements MailTransport {
         subject,
         text: text,
       });
-    }
-    catch (error: Error) {
-      console.error(
-        `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
-      );
-      throw new EmailError(error);
+    } catch (error) {
+      if (error instanceof Error) {
+        console.error(
+          `Failed to send email to ${to}, ${subject}. Error ${error.name}: ${error.message}`
+        );
+        throw new EmailError(error);
+      } else {
+        throw error;
+      }
     }
   }
 }