🔀 Merge pull request #21 from Importantus/hotfix-axios-backend-url

🐛 Fix: the new backend url does not get set as axios base url
Importantus · Feb 3, 2024 · 6679db6 · 6679db6
2 parents e895715 + 3cf64c0
commit 6679db6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/frontend/src/axios.ts b/frontend/src/axios.ts
@@ -16,6 +16,7 @@ instance.interceptors.request.use((config) => {
     if (authStore.isLoggedIn) {
         config.headers.Authorization = `Bearer ${authStore.token}`
     }
+    config.baseURL = settingsStore.getBackendUrl()
 
     return config
 }, (error) => {

diff --git a/frontend/src/stores/settings.ts b/frontend/src/stores/settings.ts
@@ -33,7 +33,6 @@ export const useSettingsStore = defineStore({
             return undefined;
         },
         getRawBackendUrl(): string | undefined {
-            console.log("Using cached backend url");
             return this.settings.backendUrl;
         },
         async fetchBackendUrl(): Promise<string> {
@@ -45,12 +44,9 @@ export const useSettingsStore = defineStore({
             return loadedUrl;
         },
         setBackendUrl(url: string) {
-            const changed = this.settings.backendUrl !== url;
-            if (changed) {
-                this.settings.backendUrl = url;
-                this.updateSettings();
-                configuredAxios.defaults.baseURL = prepareBackendUrl(url);
-            }
+            this.settings.backendUrl = url;
+            this.updateSettings();
+            configuredAxios.defaults.baseURL = prepareBackendUrl(url);
         },
         applySettings() {
             this.applyFontFactor();