blackteay/fes.js
1
0
Fork 0
mirror of https://github.com/WeBankFinTech/fes.js.git synced 2025-04-06 03:59:53 +08:00
Code Issues Projects Releases Wiki Activity

Merge pull request #78 from WeBankFinTech/fix_request_key

Browse Source 
Fix request key
This commit is contained in:
aring 2021-11-18 14:10:23 +08:00 committed by GitHub
commit 2ae1633de4
2 changed files with 44 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
packages/fes-plugin-request/src/template/genRequestKey.js
View File

@ -5,11 +5,20 @@ import { isURLSearchParams } from './helpers';
* 一个请求同时包含 data | params 参数的设计本身不合理
* 不对这种情况进行兼容
*/
export default async function genRequestKey(ctx, next) {
const { url, data, method } = ctx.config;
const getQueryString = (data) => {
if (isURLSearchParams(data)) {
ctx.key = `${url}${data.toString()}${method}`;
return data.toString();
}
ctx.key = `${url}${JSON.stringify(data)}${method}`;
return data ? JSON.stringify(data) : '';
};
export default async function genRequestKey(ctx, next) {
const {
url, data, params, method
} = ctx.config;
ctx.key = `${url}${getQueryString(data)}${getQueryString(params)}${method}`;
await next();
}

42
packages/fes-template-h5/src/pages/index.vue
View File

@ -46,17 +46,37 @@ export default {
// }).then((res) => {
// console.log(res);
// });
request('/api', null, {
mergeRequest: true
}).then((res) => {
console.log(res);
});
request('/api', null, {
throttle: 3000,
cache: true
}).then((res) => {
console.log(res);
});
// request('/api', null, {
// mergeRequest: true
// }).then((res) => {
// console.log(res);
// });
// request('/api', null, {
// throttle: 3000,
// cache: true
// }).then((res) => {
// console.log(res);
// });
const get = (id) => {
request('/get/api', { id }, {
method: 'get'
});
};
const post = (id) => {
request('/get/api', { id }, {
});
};
get(1);
get(2);
get(3);
post(1);
post(2);
post(3);
// setTimeout(() => {
// request('/api', null, {