From 4b72bbfc7f2d755a95c905e265b37b2de7e4a682 Mon Sep 17 00:00:00 2001
From: M A E R Y O <pu8046@gmail.com>
Date: Mon, 31 Jul 2023 20:27:24 +0900
Subject: [PATCH] Remove redundant calls to baseURL/url #2581 (#2579)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---------

Co-authored-by: 이광오 <maeryo@hanwha.com>
Co-authored-by: Oleg Lobanov <oleg.lobanov@bitvavo.com>
---
 frontend/src/api/files.js       |  1 -
 frontend/src/api/tus.js         | 21 +++++++++++++--------
 frontend/src/api/utils.js       |  1 -
 frontend/src/utils/constants.js |  2 +-
 4 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/frontend/src/api/files.js b/frontend/src/api/files.js
index fbe15a7375..1c2ea80a47 100644
--- a/frontend/src/api/files.js
+++ b/frontend/src/api/files.js
@@ -88,7 +88,6 @@ export async function post(url, content = "", overwrite = false, onupload) {
       !["http:", "https:"].includes(window.location.protocol)) ||
     // Tus is disabled / not applicable
     !(await useTus(content));
-
   return useResourcesApi
     ? postResources(url, content, overwrite, onupload)
     : postTus(url, content, overwrite, onupload);
diff --git a/frontend/src/api/tus.js b/frontend/src/api/tus.js
index 47e443b465..bda4f40c31 100644
--- a/frontend/src/api/tus.js
+++ b/frontend/src/api/tus.js
@@ -1,5 +1,5 @@
 import * as tus from "tus-js-client";
-import { tusEndpoint, tusSettings } from "@/utils/constants";
+import { baseURL, tusEndpoint, tusSettings } from "@/utils/constants";
 import store from "@/store";
 import { removePrefix } from "@/api/utils";
 import { fetchURL } from "./utils";
@@ -7,20 +7,25 @@ import { fetchURL } from "./utils";
 const RETRY_BASE_DELAY = 1000;
 const RETRY_MAX_DELAY = 20000;
 
-export async function upload(url, content = "", overwrite = false, onupload) {
+export async function upload(
+  filePath,
+  content = "",
+  overwrite = false,
+  onupload
+) {
   if (!tusSettings) {
     // Shouldn't happen as we check for tus support before calling this function
     throw new Error("Tus.io settings are not defined");
   }
 
-  url = removePrefix(url);
-  let resourceUrl = `${tusEndpoint}${url}?override=${overwrite}`;
+  filePath = removePrefix(filePath);
+  let resourcePath = `${tusEndpoint}${filePath}?override=${overwrite}`;
 
-  await createUpload(resourceUrl);
+  await createUpload(resourcePath);
 
   return new Promise((resolve, reject) => {
     let upload = new tus.Upload(content, {
-      uploadUrl: resourceUrl,
+      uploadUrl: `${baseURL}${resourcePath}`,
       chunkSize: tusSettings.chunkSize,
       retryDelays: computeRetryDelays(tusSettings),
       parallelUploads: 1,
@@ -46,8 +51,8 @@ export async function upload(url, content = "", overwrite = false, onupload) {
   });
 }
 
-async function createUpload(resourceUrl) {
-  let headResp = await fetchURL(resourceUrl, {
+async function createUpload(resourcePath) {
+  let headResp = await fetchURL(resourcePath, {
     method: "POST",
   });
   if (headResp.status !== 201) {
diff --git a/frontend/src/api/utils.js b/frontend/src/api/utils.js
index 87d6a243ab..7883c4ceca 100644
--- a/frontend/src/api/utils.js
+++ b/frontend/src/api/utils.js
@@ -8,7 +8,6 @@ export async function fetchURL(url, opts, auth = true) {
   opts.headers = opts.headers || {};
 
   let { headers, ...rest } = opts;
-
   let res;
   try {
     res = await fetch(`${baseURL}${url}`, {
diff --git a/frontend/src/utils/constants.js b/frontend/src/utils/constants.js
index b4212a0867..2dbe44a1a1 100644
--- a/frontend/src/utils/constants.js
+++ b/frontend/src/utils/constants.js
@@ -17,7 +17,7 @@ const resizePreview = window.FileBrowser.ResizePreview;
 const enableExec = window.FileBrowser.EnableExec;
 const tusSettings = window.FileBrowser.TusSettings;
 const origin = window.location.origin;
-const tusEndpoint = `${baseURL}/api/tus`;
+const tusEndpoint = `/api/tus`;
 
 export {
   name,