staleIfError: true should use the stale-if-error directive if present

[arthurfiorette]

Describe the bug

When using staleIfError: true, its time is resolved as forever, but if the cache-control header included a stale-if-error directive, that time should be used instead.

To Reproduce

axios.get('url to a server that returns stale-if-error', {
  cache: { staleIfError: true }
});

// The cache will be stale forever.

Expected behavior

When staleIfError is true and the server returned the stale-if-error directive, it should behave exactly the same as staleIfError: <ms in the stale-if-error directive>

Additional context

• Axios: v1.3.1
• Axios Cache Interceptor: v1.0.1

[arthurfiorette]

Anyone up to working on this?

[gmazoni]

Working on

[arthurfiorette]

Whoopsie, can you change the documentation where it was affected?

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

Line 167 in ee8b6b3

* - `boolean` -> `false` disables and `true` enables with infinite time.

and

https://github.com/arthurfiorette/axios-cache-interceptor/blob/main/docs/src/config/request-specifics.md#cachestaleiferror