Merge pull request #197 from hypercerts-org/fix/email-log-field

fix(auth-service): log email recipients consistently

Author: aspiers

.changeset/email-log-recipient-field.md

@@ -0,0 +1,9 @@
+---
+'ePDS': minor
+---
+
+Email delivery logs use a consistent recipient field.
+
+**Affects:** Operators
+
+**Operators:** All four email delivery paths now log the recipient as `email`: client-branded OTP, standard sign-in OTP, welcome OTP, and backup-email verification. Update structured-log queries for `Sent client-branded OTP email` to read `email` instead of `to`.

packages/auth-service/src/email/sender.ts

@@ -133,7 +133,7 @@ export class EmailSender {
           html: branded.html,
         })
         logger.info(
-          { to, clientId: opts.clientId },
+          { email: to, clientId: opts.clientId },
           'Sent client-branded OTP email',
         )
         return
@@ -165,6 +165,7 @@ export class EmailSender {
       text,
       html,
     })
+    logger.info({ email: to }, 'Sent sign-in OTP email')
   }
 
   private async sendWelcomeCode(opts: {
@@ -183,6 +184,7 @@ export class EmailSender {
       text,
       html,
     })
+    logger.info({ email: to }, 'Sent welcome OTP email')
   }
 
   async sendBackupEmailVerification(opts: {
@@ -201,5 +203,6 @@ export class EmailSender {
       text,
       html,
     })
+    logger.info({ email: to }, 'Sent backup email verification')
   }
 }