Optimization: Reduce cache calls about axios-cache-interceptor

[arthurfiorette]

Hey @paescuj is this directly impacting the performance of your app? Request and response in axios should have completely separated state from each other, that's why two finds happen.

I do not have time or a appealing reason to do such optimization this storage access should be almost irrelevant comparing to a network request which would happen always without axios-cache-interceptor.

However, I'm happy to guide you if you wanna work on it.

Let me walk you through what is currently happening in your usecase.

1. Request is made, get() is called in the request interceptor.

axios-cache-interceptor/src/interceptors/request.ts

Line 47 in d4fa7d8

let cache = await axios.storage.get(config.id, config);

2. The storage wrapper detects the storage returned an stale entry with cached status, set() is called to also update this entry in the db.

axios-cache-interceptor/src/storage/build.ts

Line 131 in d4fa7d8

await set(key, value, config);

3. Everything is loaded to wait for a network response, a set() call is made to update this entry in the db to loading.

axios-cache-interceptor/src/interceptors/request.ts

Lines 92 to 95 in d4fa7d8

	await axios.storage.set(
	config.id,
	{
	state: 'loading',

4. Request is sent to network, event 'Sending request, waiting for response' is emitted.

5. Response arrives, the response interceptor has no real information of what happened before and has no state connection to the request interceptor, so another get() is performed.

axios-cache-interceptor/src/interceptors/response.ts

Line 93 in d4fa7d8

const cache = await axios.storage.get(response.id, config);

6. Response is processed and before returning, a set() is called to insert the response gotten from network.

axios-cache-interceptor/src/interceptors/response.ts

Line 219 in d4fa7d8

await axios.storage.set(response.id, newCache, config);

---

However the use case you presented here probably have the highest ratio of db calls per request. Cached requests should have only a single get().