From 471237be92d24fe046ff8e6bb627cd09ac2bf8ea Mon Sep 17 00:00:00 2001 From: zRain <2296342883@qq.com> Date: Thu, 30 May 2024 12:07:53 +0800 Subject: [PATCH] [KYUUBI #6428] kyuubi-server/web-ui should not to set a timeout for axios requests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit # :mag: Description ## Issue References ๐Ÿ”— This pull request fixes #6428 ## Describe Your Solution ๐Ÿ”ง Remove axios request timeout limitation. ## Types of changes :bookmark: - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan ๐Ÿงช #### Behavior Without This Pull Request :coffin: Ref to #6428 description. #### Behavior With This Pull Request :tada: Get result after a period of time. #### Related Unit Tests kyuubi-server/web-ui: `pnpm test` all passed. --- # Checklist ๐Ÿ“ - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6429 from zRains/remove-axios-timeout. Closes #6428 e947deabe [zRain] fix: remove timeout Authored-by: zRain <2296342883@qq.com> Signed-off-by: Cheng Pan --- kyuubi-server/web-ui/src/utils/request.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kyuubi-server/web-ui/src/utils/request.ts b/kyuubi-server/web-ui/src/utils/request.ts index 3b5b9c34b..5dd9cae35 100644 --- a/kyuubi-server/web-ui/src/utils/request.ts +++ b/kyuubi-server/web-ui/src/utils/request.ts @@ -20,9 +20,8 @@ import { useAuthStore } from '@/pinia/auth/auth' // create an axios instance const service = axios.create({ - baseURL: '/', // url = base url + request url + baseURL: '/' // url = base url + request url // withCredentials: true, // send cookies when cross-domain requests - timeout: 60000 // request timeout }) // request interceptor