From 31eb67efb11b0addabde2ffc51f3650f08ca551d Mon Sep 17 00:00:00 2001 From: Hongbo Wu Date: Fri, 17 Feb 2023 22:05:35 +0800 Subject: [PATCH] Fix sometimes there is extra spaces between sender and email address by checking only the email address --- .../content-handler/src/newsletters/india-times-handler.ts | 5 ++++- packages/content-handler/test/newsletter.test.ts | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/content-handler/src/newsletters/india-times-handler.ts b/packages/content-handler/src/newsletters/india-times-handler.ts index 07c1d35b7..58a59815f 100644 --- a/packages/content-handler/src/newsletters/india-times-handler.ts +++ b/packages/content-handler/src/newsletters/india-times-handler.ts @@ -1,4 +1,5 @@ import { ContentHandler } from '../content-handler' +import addressparser from 'addressparser' export class IndiaTimesHandler extends ContentHandler { constructor() { @@ -13,7 +14,9 @@ export class IndiaTimesHandler extends ContentHandler { dom: Document }): Promise { return Promise.resolve( - input.from === 'The Times of India ' + addressparser(input.from).some( + (e) => e.address === 'newsletters@timesofindia.com' + ) ) } diff --git a/packages/content-handler/test/newsletter.test.ts b/packages/content-handler/test/newsletter.test.ts index 131236ffa..0ffa6d666 100644 --- a/packages/content-handler/test/newsletter.test.ts +++ b/packages/content-handler/test/newsletter.test.ts @@ -362,7 +362,7 @@ describe('Newsletter email test', () => { const html = load('./test/data/india-times-newsletter.html') const handler = await getNewsletterHandler({ html, - from: 'The Times of India ', + from: 'The Times of India ', headers: {}, }) expect(handler).to.be.instanceOf(IndiaTimesHandler)