From ef0a5b54ba6b0ad6bbb47c07fe98f5bc6137ba45 Mon Sep 17 00:00:00 2001 From: Lucas Smith Date: Thu, 5 Mar 2026 15:12:20 +1100 Subject: [PATCH] fix: verify before re-registering in email sync (#2568) --- .../internal/sync-email-domains.handler.ts | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/packages/lib/jobs/definitions/internal/sync-email-domains.handler.ts b/packages/lib/jobs/definitions/internal/sync-email-domains.handler.ts index 8d580a010..2b9768244 100644 --- a/packages/lib/jobs/definitions/internal/sync-email-domains.handler.ts +++ b/packages/lib/jobs/definitions/internal/sync-email-domains.handler.ts @@ -46,6 +46,14 @@ export const run = async ({ io }: { payload: TSyncEmailDomainsJobDefinition; io: batch.map(async (domain) => { const shouldReregister = domain.createdAt < reregisterCutoff; + const { isVerified } = await verifyEmailDomain(domain.id); + + if (isVerified) { + io.logger.info(`Domain "${domain.domain}" is verified`); + + return 'verified' as const; + } + if (shouldReregister) { io.logger.info( `Domain "${domain.domain}" has been pending since ${domain.createdAt.toISOString()}, attempting re-registration`, @@ -55,9 +63,7 @@ export const run = async ({ io }: { payload: TSyncEmailDomainsJobDefinition; io: return 'reregistered' as const; } - const { isVerified } = await verifyEmailDomain(domain.id); - - return isVerified ? ('verified' as const) : ('pending' as const); + return 'pending' as const; }), );