Skip to content

Commit c68f8db

Browse files
authored
Encode URI and escape file name in script.js
Fix security vulnerabilities
1 parent 29d1d92 commit c68f8db

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

js/script.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1121,7 +1121,8 @@ Options -Indexes
11211121
}
11221122

11231123
const addURItoURL = function(uri) {
1124-
globalThis.location.href = globalThis.location.href.split("?uri=")[0] + "?uri=" + uri;
1124+
const encodedUri = encodeURIComponent(uri);
1125+
globalThis.location.href = globalThis.location.href.split("?uri=")[0] + "?uri=" + encodedUri;
11251126
}
11261127

11271128
const getRelationURI = async function(fields, fieldValues, db) {
@@ -2712,7 +2713,7 @@ Options -Indexes
27122713
const fileExtensionInput = globalThis.document.getElementById("file-extension");
27132714
fileExtensionInput.value = fileExt;
27142715
const selectedFile = globalThis.document.getElementById("selected-media-file");
2715-
selectedFile.innerHTML = "<div id='selected-file-text'>Selected file:</div> <div id='filename'>" + file.name + "</div><a id='remove-media-file' class='slim-button'>Remove it</a>";
2716+
selectedFile.innerHTML = "<div id='selected-file-text'>Selected file:</div> <div id='filename'>" + escapeHTML(file.name) + "</div><a id='remove-media-file' class='slim-button'>Remove it</a>";
27162717
const removeMediaFile = globalThis.document.getElementById("remove-media-file");
27172718
removeMediaFile.addEventListener("click", removeMediaFileHandler);
27182719
const dropZoneText = globalThis.document.getElementById("drop-zone-text");

0 commit comments

Comments
 (0)