From d58d605ee9d00a28aa39a6e2efcbf849a56f48f9 Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Mon, 14 Jul 2025 14:00:40 +0530
Subject: [PATCH 1/8] Add project id for post requests as well in the params

---
 ui/src/utils/request.js | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/ui/src/utils/request.js b/ui/src/utils/request.js
index 7c757691f2b3..aa3ee3312d7f 100644
--- a/ui/src/utils/request.js
+++ b/ui/src/utils/request.js
@@ -149,6 +149,8 @@ const err = (error) => {
 service.interceptors.request.use(config => {
   source = sourceToken.getSource()
   config.cancelToken = source.token
+
+  // Handle GET requests (params)
   if (config && config.params) {
     config.params.response = 'json'
     const project = vueProps.$localStorage.get(CURRENT_PROJECT)
@@ -163,6 +165,26 @@ service.interceptors.request.use(config => {
       config.params.ignoreproject = null
     }
   }
+
+  // Handle POST requests (data)
+  if (config && config.data && config.data instanceof URLSearchParams) {
+    const project = vueProps.$localStorage.get(CURRENT_PROJECT)
+    const command = config.data.get('command')
+    const hasProjectId = config.data.has('projectid')
+    const ignoreProject = config.data.has('ignoreproject')
+
+    if (!hasProjectId && !ignoreProject && project && project.id) {
+      if (command === 'listTags') {
+        config.data.append('projectid', '-1')
+      } else if (command !== 'assignVirtualMachine') {
+        config.data.append('projectid', project.id)
+      }
+    }
+    if (config.data.has('ignoreproject')) {
+      config.data.delete('ignoreproject')
+    }
+  }
+
   return config
 }, err)
 

From 70cc55d3e41122a7ab635c3b6b07c5df8bfc7c22 Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Mon, 14 Jul 2025 14:04:24 +0530
Subject: [PATCH 2/8] Replace leftover api calls to getAPI calls

---
 ui/src/components/view/ImageDeployInstanceButton.vue | 6 +++---
 ui/src/views/compute/KubernetesAddNodes.vue          | 6 +++---
 ui/src/views/compute/KubernetesRemoveNodes.vue       | 4 ++--
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/ui/src/components/view/ImageDeployInstanceButton.vue b/ui/src/components/view/ImageDeployInstanceButton.vue
index 4f632cc03836..b2d4b55bc6a3 100644
--- a/ui/src/components/view/ImageDeployInstanceButton.vue
+++ b/ui/src/components/view/ImageDeployInstanceButton.vue
@@ -41,7 +41,7 @@
 </template>
 
 <script>
-import { api } from '@/api'
+import { getAPI } from '@/api'
 import ResourceIcon from '@/components/view/ResourceIcon'
 
 export default {
@@ -100,7 +100,7 @@ export default {
       this.itemCount = 0
       this.fetchLoading = true
       this.zones = []
-      api(this.imageApi, params).then(json => {
+      getAPI(this.imageApi, params).then(json => {
         const imageResponse = json?.[this.imageApi.toLowerCase() + 'response']?.[this.$route.meta.name] || []
         this.zones = imageResponse.map(i => ({
           id: i.zoneid,
@@ -122,7 +122,7 @@ export default {
       }
       const zoneids = this.zones.map(z => z.id)
       this.loading = true
-      api('listZones', { showicon: true, ids: zoneids.join(',') }).then(json => {
+      getAPI('listZones', { showicon: true, ids: zoneids.join(',') }).then(json => {
         this.zones = json.listzonesresponse.zone || []
       }).finally(() => {
         this.loading = false
diff --git a/ui/src/views/compute/KubernetesAddNodes.vue b/ui/src/views/compute/KubernetesAddNodes.vue
index dddcc90be252..422197bc7c75 100644
--- a/ui/src/views/compute/KubernetesAddNodes.vue
+++ b/ui/src/views/compute/KubernetesAddNodes.vue
@@ -66,7 +66,7 @@
 
 <script>
 import { ref, reactive, toRaw } from 'vue'
-import { api } from '@/api'
+import { getAPI } from '@/api'
 import TooltipLabel from '@/components/widgets/TooltipLabel'
 
 export default {
@@ -112,7 +112,7 @@ export default {
     callListVms (accountId, domainId) {
       return new Promise((resolve) => {
         this.volumes = []
-        api('listVirtualMachines', {
+        getAPI('listVirtualMachines', {
           accountId: accountId,
           domainId: domainId,
           details: 'min',
@@ -172,7 +172,7 @@ export default {
     },
     addNodesToKubernetesCluster (params) {
       return new Promise((resolve, reject) => {
-        api('addNodesToKubernetesCluster', params).then(json => {
+        getAPI('addNodesToKubernetesCluster', params).then(json => {
           const jobId = json.addnodestokubernetesclusterresponse.jobid
           return resolve(jobId)
         }).catch(error => {
diff --git a/ui/src/views/compute/KubernetesRemoveNodes.vue b/ui/src/views/compute/KubernetesRemoveNodes.vue
index 57d6409b7841..a412dbce3fc0 100644
--- a/ui/src/views/compute/KubernetesRemoveNodes.vue
+++ b/ui/src/views/compute/KubernetesRemoveNodes.vue
@@ -54,7 +54,7 @@
 
 <script>
 import { ref, reactive, toRaw } from 'vue'
-import { api } from '@/api'
+import { getAPI } from '@/api'
 import TooltipLabel from '@/components/widgets/TooltipLabel'
 
 export default {
@@ -137,7 +137,7 @@ export default {
     },
     removeNodesFromKubernetesCluster (params) {
       return new Promise((resolve, reject) => {
-        api('removeNodesFromKubernetesCluster', params).then(json => {
+        getAPI('removeNodesFromKubernetesCluster', params).then(json => {
           const jobId = json.removenodesfromkubernetesclusterresponse.jobid
           return resolve(jobId)
         }).catch(error => {

From 4c55cc238a07ee50588381057d03602c810efdcd Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Mon, 14 Jul 2025 14:18:36 +0530
Subject: [PATCH 3/8] ui: don't remove values from request if the value is null
 or empty string

---
 ui/src/api/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ui/src/api/index.js b/ui/src/api/index.js
index 45938df6cab7..fbc84544b012 100644
--- a/ui/src/api/index.js
+++ b/ui/src/api/index.js
@@ -47,7 +47,7 @@ export function postAPI (command, data = {}) {
   params.append('response', 'json')
   if (data) {
     Object.entries(data).forEach(([key, value]) => {
-      if (value !== undefined && value !== null && value !== '') {
+      if (value !== undefined) {
         params.append(key, value)
       }
     })

From f8fa89b409c0eff0f13acd1b13079b9b3abbe58d Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Mon, 14 Jul 2025 15:04:36 +0530
Subject: [PATCH 4/8] Address comments

---
 ui/src/utils/request.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/ui/src/utils/request.js b/ui/src/utils/request.js
index aa3ee3312d7f..35f78ac8566c 100644
--- a/ui/src/utils/request.js
+++ b/ui/src/utils/request.js
@@ -151,6 +151,14 @@ service.interceptors.request.use(config => {
   config.cancelToken = source.token
 
   // Handle GET requests (params)
+  handleGetRequestParams(config)
+
+  handlePostRequestParams(config)
+
+  return config
+}, err)
+
+function handleGetRequestParams (config) {
   if (config && config.params) {
     config.params.response = 'json'
     const project = vueProps.$localStorage.get(CURRENT_PROJECT)
@@ -162,11 +170,12 @@ service.interceptors.request.use(config => {
       }
     }
     if (config.params.ignoreproject !== undefined) {
-      config.params.ignoreproject = null
+      delete config.params.ignoreproject
     }
   }
+}
 
-  // Handle POST requests (data)
+function handlePostRequestParams (config) {
   if (config && config.data && config.data instanceof URLSearchParams) {
     const project = vueProps.$localStorage.get(CURRENT_PROJECT)
     const command = config.data.get('command')
@@ -184,9 +193,7 @@ service.interceptors.request.use(config => {
       config.data.delete('ignoreproject')
     }
   }
-
-  return config
-}, err)
+}
 
 // response interceptor
 service.interceptors.response.use((response) => {

From 1247a1b6b944d4888a9bab07af293b44e8f7c1d7 Mon Sep 17 00:00:00 2001
From: Vishesh <vishesh92@gmail.com>
Date: Mon, 14 Jul 2025 16:20:45 +0530
Subject: [PATCH 5/8] Apply suggestions from code review

---
 ui/src/utils/request.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/ui/src/utils/request.js b/ui/src/utils/request.js
index 35f78ac8566c..42b26c9785b1 100644
--- a/ui/src/utils/request.js
+++ b/ui/src/utils/request.js
@@ -150,7 +150,6 @@ service.interceptors.request.use(config => {
   source = sourceToken.getSource()
   config.cancelToken = source.token
 
-  // Handle GET requests (params)
   handleGetRequestParams(config)
 
   handlePostRequestParams(config)

From 732332160348067f5c99cc6a6c9ad28e684af4e1 Mon Sep 17 00:00:00 2001
From: Vishesh <vishesh92@gmail.com>
Date: Tue, 15 Jul 2025 13:34:43 +0530
Subject: [PATCH 6/8] Apply suggestions from code review

Co-authored-by: Suresh Kumar Anaparti <sureshkumar.anaparti@gmail.com>
---
 ui/src/views/compute/KubernetesAddNodes.vue    | 4 ++--
 ui/src/views/compute/KubernetesRemoveNodes.vue | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/ui/src/views/compute/KubernetesAddNodes.vue b/ui/src/views/compute/KubernetesAddNodes.vue
index 422197bc7c75..2ee0b5ed5283 100644
--- a/ui/src/views/compute/KubernetesAddNodes.vue
+++ b/ui/src/views/compute/KubernetesAddNodes.vue
@@ -66,7 +66,7 @@
 
 <script>
 import { ref, reactive, toRaw } from 'vue'
-import { getAPI } from '@/api'
+import { getAPI, postAPI } from '@/api'
 import TooltipLabel from '@/components/widgets/TooltipLabel'
 
 export default {
@@ -172,7 +172,7 @@ export default {
     },
     addNodesToKubernetesCluster (params) {
       return new Promise((resolve, reject) => {
-        getAPI('addNodesToKubernetesCluster', params).then(json => {
+        postAPI('addNodesToKubernetesCluster', params).then(json => {
           const jobId = json.addnodestokubernetesclusterresponse.jobid
           return resolve(jobId)
         }).catch(error => {
diff --git a/ui/src/views/compute/KubernetesRemoveNodes.vue b/ui/src/views/compute/KubernetesRemoveNodes.vue
index a412dbce3fc0..803ec2bd1b19 100644
--- a/ui/src/views/compute/KubernetesRemoveNodes.vue
+++ b/ui/src/views/compute/KubernetesRemoveNodes.vue
@@ -54,7 +54,7 @@
 
 <script>
 import { ref, reactive, toRaw } from 'vue'
-import { getAPI } from '@/api'
+import { postAPI } from '@/api'
 import TooltipLabel from '@/components/widgets/TooltipLabel'
 
 export default {
@@ -137,7 +137,7 @@ export default {
     },
     removeNodesFromKubernetesCluster (params) {
       return new Promise((resolve, reject) => {
-        getAPI('removeNodesFromKubernetesCluster', params).then(json => {
+        postAPI('removeNodesFromKubernetesCluster', params).then(json => {
           const jobId = json.removenodesfromkubernetesclusterresponse.jobid
           return resolve(jobId)
         }).catch(error => {

From 390881577dc9eea0e84662054c79d970e54f7b07 Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Tue, 15 Jul 2025 16:21:48 +0530
Subject: [PATCH 7/8] fixup

---
 ui/src/api/index.js      | 2 +-
 ui/src/main.js           | 4 ++--
 ui/src/utils/guiTheme.js | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ui/src/api/index.js b/ui/src/api/index.js
index fbc84544b012..1f532c36336a 100644
--- a/ui/src/api/index.js
+++ b/ui/src/api/index.js
@@ -47,7 +47,7 @@ export function postAPI (command, data = {}) {
   params.append('response', 'json')
   if (data) {
     Object.entries(data).forEach(([key, value]) => {
-      if (value !== undefined) {
+      if (value !== undefined && value !== null) {
         params.append(key, value)
       }
     })
diff --git a/ui/src/main.js b/ui/src/main.js
index c25ab1066d4e..2bd9d945e9a6 100644
--- a/ui/src/main.js
+++ b/ui/src/main.js
@@ -44,7 +44,7 @@ import {
 import { VueAxios } from './utils/request'
 import directives from './utils/directives'
 import Cookies from 'js-cookie'
-import { api } from '@/api'
+import { getAPI } from '@/api'
 import { applyCustomGuiTheme } from './utils/guiTheme'
 
 vueApp.use(VueAxios, router)
@@ -106,7 +106,7 @@ fetch('config.json?ts=' + Date.now())
     let domainid = null
 
     if (userid !== undefined && Cookies.get('sessionkey')) {
-      await api('listUsers', { userid: userid }).then(response => {
+      await getAPI('listUsers', { userid: userid }).then(response => {
         accountid = response.listusersresponse.user[0].accountid
         domainid = response.listusersresponse.user[0].domainid
       })
diff --git a/ui/src/utils/guiTheme.js b/ui/src/utils/guiTheme.js
index 7cc3dc0cc034..6ca6dcbc439d 100644
--- a/ui/src/utils/guiTheme.js
+++ b/ui/src/utils/guiTheme.js
@@ -16,7 +16,7 @@
 // under the License.
 
 import { vueProps } from '@/vue-app'
-import { api } from '@/api'
+import { getAPI } from '@/api'
 
 export async function applyCustomGuiTheme (accountid, domainid) {
   await fetch('config.json').then(response => response.json()).then(config => {
@@ -45,7 +45,7 @@ export async function applyCustomGuiTheme (accountid, domainid) {
 }
 
 async function fetchGuiTheme (params) {
-  return await api('listGuiThemes', params).then(response => {
+  return await getAPI('listGuiThemes', params).then(response => {
     if (response.listguithemesresponse.guiThemes) {
       return response.listguithemesresponse.guiThemes[0]
     }

From 37d9db4020413cb65d2247a1a4457acf1bb00d48 Mon Sep 17 00:00:00 2001
From: vishesh92 <vishesh92@gmail.com>
Date: Tue, 15 Jul 2025 16:41:12 +0530
Subject: [PATCH 8/8] Return null if guiTheme requests fails

---
 ui/src/utils/guiTheme.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ui/src/utils/guiTheme.js b/ui/src/utils/guiTheme.js
index 6ca6dcbc439d..b1a7209fd270 100644
--- a/ui/src/utils/guiTheme.js
+++ b/ui/src/utils/guiTheme.js
@@ -49,6 +49,9 @@ async function fetchGuiTheme (params) {
     if (response.listguithemesresponse.guiThemes) {
       return response.listguithemesresponse.guiThemes[0]
     }
+  }).catch(error => {
+    console.error('Error fetching GUI theme:', error)
+    return null
   })
 }