diff --git a/packages/content-handler/src/content-handler.ts b/packages/content-handler/src/content-handler.ts index c7f7f1d61..ae3a81636 100644 --- a/packages/content-handler/src/content-handler.ts +++ b/packages/content-handler/src/content-handler.ts @@ -172,9 +172,9 @@ export abstract class ContentHandler { const url = (await this.parseNewsletterUrl(headers, html)) || generateUniqueUrl() const author = this.parseAuthor(from) - const unsubscribe = this.parseUnsubscribe( - headers['list-unsubscribe']?.toString() - ) + const unsubscribe = headers['list-unsubscribe'] + ? this.parseUnsubscribe(headers['list-unsubscribe'].toString()) + : undefined return { email: to, @@ -182,8 +182,8 @@ export abstract class ContentHandler { url, title: subject, author, - unsubMailTo: unsubscribe.mailTo || '', - unsubHttpUrl: unsubscribe.httpUrl || '', + unsubMailTo: unsubscribe?.mailTo || '', + unsubHttpUrl: unsubscribe?.httpUrl || '', } } } diff --git a/packages/inbound-email-handler/src/index.ts b/packages/inbound-email-handler/src/index.ts index c34d5f4e0..4ecf6fb57 100644 --- a/packages/inbound-email-handler/src/index.ts +++ b/packages/inbound-email-handler/src/index.ts @@ -189,7 +189,13 @@ export const inboundEmailHandler = Sentry.GCPFunction.wrapHttpFunction( }) res.send('newsletter received') } catch (error) { - console.log('error handling emails, will forward.', from, to, subject) + console.log( + 'error handling emails, will forward.', + from, + to, + subject, + error + ) // queue error emails await pubsub.topic(NON_NEWSLETTER_EMAIL_TOPIC).publishMessage({ json: {