Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2024-12-29 20:04:59 +08:00
Code Issues Projects Releases Packages Wiki Activity

Fix “disable images” occasionally showing images due to reuse/statefulness of Regexp.test

Browse source
This commit is contained in:
Ben Gotow 2020-04-15 13:46:48 -05:00
parent 0086b2cf13
commit 250f880040
1 changed files with 17 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
app/internal_packages/message-autoload-images/lib/autoload-images-store.ts
View file

@ -4,11 +4,7 @@ import path from 'path';
import { Utils, MessageBodyProcessor, CategoryStore } from 'mailspring-exports';
import * as AutoloadImagesActions from './autoload-images-actions';
const ImagesRegexp = /((?:src|background|placeholder|icon|background|poster|srcset)\s*=\s*['"]?(?=\w*:\/\/)|:\s*url\()+([^"')]*)/gi;
class AutoloadImagesStore extends MailspringStore {
ImagesRegexp = ImagesRegexp;
_whitelistEmails = {};
_whitelistMessageIds = {};
_whitelistEmailsPath = path.join(AppEnv.getConfigDirPath(), 'autoload-images-whitelist.txt');
@ -26,6 +22,20 @@ class AutoloadImagesStore extends MailspringStore {
});
}
getImagesRegexp = () => {
// Matches:
// - src='....'
// - background="...."
// - background: url(....)
// - background: url(""...."")
// - @import url(....)
return /((?:src|background|placeholder|icon|poster|srcset)\s*=\s*['"]?(?=[cid:|\w*:\/\/])|(?::|@import)\s*url\(['"]?)+([^"')]*)/gi;
};
getLinkTagRegexp = () => {
return /<link [^>]+>/gi;
};
shouldBlockImagesIn = message => {
const spam = CategoryStore.getSpamCategory(message.accountId);
const spamFolderId = spam ? spam.id : undefined;
@ -40,7 +50,9 @@ class AutoloadImagesStore extends MailspringStore {
return false;
}
return ImagesRegexp.test(message.body);
const containsImages = this.getImagesRegexp().test(message.body);
const containsLinkTags = this.getLinkTagRegexp().test(message.body);
return containsImages || containsLinkTags;
};
_loadWhitelist = () => {