Fix vulnerability

thingbomb · Sep 11, 2024 · c60ec98 · c60ec98
1 parent a2f346c
commit c60ec98
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/css/home.css b/src/css/home.css
@@ -138,7 +138,7 @@ header {
         max-width: 100%;
     }
 
-    .feed-item img {
+    .feed-item .icon {
         border-radius: 6px;
         height: 20px;
     }

diff --git a/src/js/home.js b/src/js/home.js
@@ -43,6 +43,10 @@ function areDatesOnTheSameDay(date1, date2) {
         date1Parts.day === date2Parts.day;
 }
 
+function removeEscapeCharacters(str) {
+    return str.replaceAll("\\", "");
+}
+
 let feedsFailedToLoad = 0;
 
 function fetchFeed(url) {
@@ -85,9 +89,9 @@ function fetchFeed(url) {
                 const span = document.createElement("span");
                 span.textContent = feedTitle;
 
-                const icon = document.createElement("img");
-                icon.src = `https://logo.clearbit.com/${url.split("/")[2]}`;
-                icon.height = 24;
+                const icon = document.createElement("img")
+                icon.src = `https://logo.clearbit.com/${encodeURIComponent(url.split("/")[2])}`
+                icon.className = "icon"
 
                 button.appendChild(icon);
                 button.appendChild(spacer);