-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: create request #11341
feat: create request #11341
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: +119 B (0%) Total Size: 10.1 MB
ℹ️ View Unchanged
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #11341 +/- ##
=======================================
Coverage 29.03% 29.03%
=======================================
Files 484 484
Lines 14667 14667
Branches 3460 3460
=======================================
Hits 4259 4259
Misses 9657 9657
Partials 751 751
☔ View full report in Codecov by Sentry. |
感觉这个需求没有意义,让整个文件变的不够清真易解了,因为用户如果有多套不同的 config ,自行再封装一层就可以: request('/api/user/list', {
// ↓ custom axios config
baseURL: '/ppp',
headers: {
'x-header': '1'
}
}) 完全没必要 |
@sherry007 你试试,为什么不能这么处理? |
@fz6m 这么用会有影响的,还是一个单例,文件中只是调整了一下代码,并没有什么影响,还有我不理解“清真”这个词 |
这样可就太麻烦了,每个请求都要写一大段配置,而且有些配置又是一样的;另外,由于我们中台服务的不同(现在有几十个了),当初定义的接口返回数据并不是很统一,所以可能需要特殊处理的得有五六种配置,如图所示(当然,历史原因,由于项目经由许多不同的人,我后面会去进行优化,配置可能会减少一些) p.s 我还是觉得多一个 create 方法不是多此一举,我觉得项目越来越大之后,接入的中台服务越来越多是极有可能导致需要不同的 request instance 接发请求的 |
感谢!!!ღ( ´・ᴗ・` )比心 |
|
@fz6m 的观点,我觉得是有道理的; |
我觉得你这种多套前缀baseUrl或者请求和响应都是多套的,可以直接 |
特殊情况下,如何使用request,忽略全局拦截器? |
plugin-request 支持多请求实例
有时候由于业务需要,我们需要在同一个项目中使用两套请求配置。见 11305。
此处提供
createAxios, requestHandle
保证同一个项目中创建出来的请求实例保持一致的逻辑,差异仅仅是配置不同。使用方法如下所示:
Closes: #11305