RevelaNetwork/Revela-v4
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix bug with data urls attempting to route to the favicon proxy.

Browse source
This commit is contained in:
wearrrrr 2024-04-01 13:48:57 -05:00
parent e54858620b
commit f5fdaf32e6
1 changed files with 12 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/components/ProxyRegistrar.astro
View file

@ -227,15 +227,21 @@
(iframe.contentDocument.querySelector("link[rel='icon']") as HTMLLinkElement) ||
(iframe.contentDocument.querySelector("link[rel*='icon']") as HTMLLinkElement);
if (favicon && favicon.href) {
if (
proxiedFavicon.src ==
`${window.location.origin}/custom-favicon?url=${encodeURIComponent(favicon.href)}` ||
hasErrored
)
let encodedHREF = encodeURIComponent(favicon.href);
if (favicon.href.includes("data:image")) {
proxiedFavicon.src = favicon.href;
return;
}
if (proxiedFavicon.src == `${window.location.origin}/custom-favicon?url=${encodedHREF}`)
return;
}
if (favicon) {
proxiedFavicon.src = `/custom-favicon?url=${encodeURIComponent(favicon.href)}`;
let encodedHREF = encodeURIComponent(favicon.href);
if (favicon.href.includes("data:image")) {
proxiedFavicon.src = favicon.href;
return;
}
proxiedFavicon.src = `/custom-favicon?url=${encodedHREF}`;
proxiedFavicon.addEventListener("error", () => {
proxiedFavicon.src = "/favicon.ico";
});