fix: uri encode fs delete

Signed-off-by: Evan Song <theevansong@gmail.com>
modrinth · Oct 18, 2024 · fe541d9 · fe541d9
1 parent c24e8a3
commit fe541d9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/apps/frontend/src/composables/pyroServers.ts b/apps/frontend/src/composables/pyroServers.ts
@@ -719,8 +719,9 @@ const moveFileOrFolder = (path: string, newPath: string) => {
 };
 
 const deleteFileOrFolder = (path: string, recursive: boolean) => {
+  const encodedPath = encodeURIComponent(path);
   return retryWithAuth(async () => {
-    return await PyroFetch(`/delete?path=${path}&recursive=${recursive}`, {
+    return await PyroFetch(`/delete?path=${encodedPath}&recursive=${recursive}`, {
       method: "DELETE",
       override: internalServerRefrence.value.fs.auth,
     });

diff --git a/apps/frontend/src/pages/servers/manage/[id]/files.vue b/apps/frontend/src/pages/servers/manage/[id]/files.vue
@@ -750,10 +750,8 @@ const downloadFile = async (item: any) => {
 
 const deleteItem = async () => {
   try {
-    await props.server.fs?.deleteFileOrFolder(
-      `${currentPath.value}/${selectedItem.value.name}`.replace("//", "/"),
-      selectedItem.value.type === "directory",
-    );
+    const path = `${currentPath.value}/${selectedItem.value.name}`.replace("//", "/");
+    await props.server.fs?.deleteFileOrFolder(path, selectedItem.value.type === "directory");
     currentPage.value = 1;
     items.value = [];
     await fetchData();