fix: update snapshots, fix pgp edge case for optimization

forwardemail · Jan 2, 2025 · 6bece4e · 6bece4e
1 parent 0b8ca22
commit 6bece4e
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 7 deletions.
diff --git a/helpers/send-email.js b/helpers/send-email.js
@@ -152,6 +152,7 @@ async function getPGPResults({
   return { finalRaw, pgp };
 }
 
+// eslint-disable-next-line complexity
 async function sendEmail(
   {
     session,
@@ -236,7 +237,7 @@ async function sendEmail(
   // - pm.me
   // - + supports custom domains since we use truthSource (resolved MX server)
   //
-  if (truthSource === 'protonmail.ch') {
+  if (pgpResults.pgp && truthSource === 'protonmail.ch') {
     pgpResults.finalRaw = await signMessage(raw, domain);
     pgpResults.pgp = false;
     // TODO: email users one-time per month regarding this issue

diff --git a/test/web/snapshots/index.js.md b/test/web/snapshots/index.js.md
diff --git a/test/web/snapshots/index.js.snap b/test/web/snapshots/index.js.snap
diff --git a/test/web/snapshots/otp.js.md b/test/web/snapshots/otp.js.md
diff --git a/test/web/snapshots/otp.js.snap b/test/web/snapshots/otp.js.snap