Moved handling of images using proxy in preperation of #491

This commit is contained in:
the-djmaze 2022-08-26 11:11:19 +02:00
parent f41559ae2f
commit aff2d4dd76
2 changed files with 56 additions and 51 deletions

View file

@ -39,7 +39,6 @@ export const
cleanHtml = (html, oAttachments, removeColors) => { cleanHtml = (html, oAttachments, removeColors) => {
const const
debug = false, // Config()->Get('debug', 'enable', false); debug = false, // Config()->Get('debug', 'enable', false);
useProxy = !!SettingsGet('UseLocalProxyForExternalImages'),
detectHiddenImages = true, // !!SettingsGet('try_to_detect_hidden_images'), detectHiddenImages = true, // !!SettingsGet('try_to_detect_hidden_images'),
result = { result = {
@ -229,10 +228,9 @@ export const
value = getAttribute('src'); value = getAttribute('src');
delAttribute('src'); delAttribute('src');
if ('IMG' === name) {
let attachment; let attachment;
if (detectHiddenImages if (detectHiddenImages
&& 'IMG' === name
&& (('' != getAttribute('height') && 3 > pInt(getAttribute('height'))) && (('' != getAttribute('height') && 3 > pInt(getAttribute('height')))
|| ('' != getAttribute('width') && 3 > pInt(getAttribute('width'))) || ('' != getAttribute('width') && 3 > pInt(getAttribute('width')))
|| [ || [
@ -265,7 +263,7 @@ export const
} }
else if (/^(https?:)?\/\//i.test(value)) else if (/^(https?:)?\/\//i.test(value))
{ {
setAttribute('data-x-src', useProxy ? proxy(value) : value); setAttribute('data-x-src', value);
result.hasExternals = true; result.hasExternals = true;
} }
else if ('data:image/' === value.slice(0, 11)) else if ('data:image/' === value.slice(0, 11))
@ -277,6 +275,11 @@ export const
setAttribute('data-x-broken-src', value); setAttribute('data-x-broken-src', value);
} }
} }
else
{
setAttribute('data-x-broken-src', value);
}
}
if (hasAttribute('background')) { if (hasAttribute('background')) {
oStyle.backgroundImage = 'url("' + getAttribute('background') + '")'; oStyle.backgroundImage = 'url("' + getAttribute('background') + '")';
@ -323,7 +326,7 @@ export const
} }
} else if (/^(https?:)?\/\//.test(lowerUrl)) { } else if (/^(https?:)?\/\//.test(lowerUrl)) {
result.hasExternals = true; result.hasExternals = true;
urls_remote.push([property, useProxy ? proxy(found) : value]); urls_remote.push([property, found]);
} else if ('data:image/' === lowerUrl.slice(0, 11)) { } else if ('data:image/' === lowerUrl.slice(0, 11)) {
oStyle[property] = value; oStyle[property] = value;
} else { } else {

View file

@ -3,10 +3,10 @@ import ko from 'ko';
import { MessagePriority } from 'Common/EnumsUser'; import { MessagePriority } from 'Common/EnumsUser';
import { i18n } from 'Common/Translator'; import { i18n } from 'Common/Translator';
import { doc } from 'Common/Globals'; import { doc, SettingsGet } from 'Common/Globals';
import { encodeHtml, plainToHtml, cleanHtml } from 'Common/Html'; import { encodeHtml, plainToHtml, cleanHtml } from 'Common/Html';
import { isArray, arrayLength, forEachObjectEntry } from 'Common/Utils'; import { isArray, arrayLength, forEachObjectEntry } from 'Common/Utils';
import { serverRequestRaw } from 'Common/Links'; import { serverRequestRaw, proxy } from 'Common/Links';
import { FolderUserStore } from 'Stores/User/Folder'; import { FolderUserStore } from 'Stores/User/Folder';
import { SettingsUserStore } from 'Stores/User/Settings'; import { SettingsUserStore } from 'Stores/User/Settings';
@ -606,16 +606,18 @@ export class MessageModel extends AbstractModel {
this.hasImages(false); this.hasImages(false);
body.rlHasImages = false; body.rlHasImages = false;
let attr = 'data-x-src'; let attr = 'data-x-src',
body.querySelectorAll('[' + attr + ']').forEach(node => { src, useProxy = !!SettingsGet('UseLocalProxyForExternalImages');
if (node.matches('img')) { body.querySelectorAll('img[' + attr + ']').forEach(node => {
node.loading = 'lazy'; node.loading = 'lazy';
} src = node.getAttribute(attr);
node.src = node.getAttribute(attr); node.src = useProxy ? proxy(src) : src;
}); });
body.querySelectorAll('[data-x-style-url]').forEach(node => { body.querySelectorAll('[data-x-style-url]').forEach(node => {
JSON.parse(node.dataset.xStyleUrl).forEach(data => node.style[data[0]] = "url('" + data[1] + "')"); JSON.parse(node.dataset.xStyleUrl).forEach(data =>
node.style[data[0]] = "url('" + (useProxy ? proxy(data[1]) : data[1]) + "')"
);
}); });
} }
} }