You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes you're right, and I think it would be an enhancement to autodelete expired cache entries so they don't hang around taking up disk space.
It would have to be done in a way that works across process restarts and multiple processes, so it couldn't just be a setTimeout() (like it is in MemoryCache). Perhaps a cache entry that contains an array of expiring entries and their expiration time, and poll that, although that array could become large and inefficient to parse into memory. If done that way, it might have to be optional behavior.
It might make sense to just have it be a feature of a new SQLiteCache or such, where the storage format can support efficiently querying for expired entries. Not sure if there's a good way to do that with cacache which is the backing store for FileSystemCache.
It seems like with a FileSystemCache, expired caches are still perisisted on disk unless the cache key is overwritten by a fresher value?
is this by design? or would it be a enhancement to autodelete expired cache entries?
Took a cursory glance at https://github.com/mistval/node-fetch-cache/blob/master/src/classes/caching/file_system_cache.ts#L23 and it seems to corroborate with my experience
The text was updated successfully, but these errors were encountered: