teambition-sdk icon indicating copy to clipboard operation
teambition-sdk copied to clipboard

handleRequestCache 缓存策略:是否只缓存状态为 200 的请求,而不是 [200, 400)

Open AmosJin opened this issue 5 years ago • 1 comments

场景:使用 sdk.lift() 接口,如果其中 request 是个返回状态为 204 的请求,那么这个 handleRequestCache 中的缓存策略是缓存的(因为它本身不关心是否是个请求,只要请求状态是 [200, 400) 就表示成功,然后 do(() => this.requestMap.set(cacheKey, true)) 缓存。 疑问:是否考虑只有 200 的才 do(() => this.requestMap.set(cacheKey, true))?

AmosJin avatar Mar 22 '19 07:03 AmosJin

@AmosJin 谢谢!我们私下讨论一下具体场景吧,因为理论上 204 返回状态一般对应更改型的 HTTP 方法,如 PUT/PATCH,而我们使用缓存策略的代码分支只针对 GET 有效,两者不应该有交互 :ok_hand:

chuan6 avatar Mar 25 '19 00:03 chuan6