Skip to content

Commit 266b81a

Browse files
zRainspan3793
authored andcommitted
[KYUUBI #6428] kyuubi-server/web-ui should not to set a timeout for axios requests
# 🔍 Description ## Issue References 🔗 This pull request fixes #6428 ## Describe Your Solution 🔧 Remove axios request timeout limitation. ## Types of changes 🔖 - [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 ⚰️ Ref to #6428 description. #### Behavior With This Pull Request 🎉 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 e947dea [zRain] fix: remove timeout Authored-by: zRain <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
1 parent b1537be commit 266b81a

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

kyuubi-server/web-ui/src/utils/request.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,8 @@ import { useAuthStore } from '@/pinia/auth/auth'
2020

2121
// create an axios instance
2222
const service = axios.create({
23-
baseURL: '/', // url = base url + request url
23+
baseURL: '/' // url = base url + request url
2424
// withCredentials: true, // send cookies when cross-domain requests
25-
timeout: 60000 // request timeout
2625
})
2726

2827
// request interceptor

0 commit comments

Comments
 (0)