Cache entry must specify a value for Size

This should be a fairly easy one to knock out, but I’m not seeing a way to set the ‘Size()’ of an item for the MemoryCacheRateLimitCounterStore and other MemoryCache* providers, for when memory cache’s SizeLimit is set.

 System.InvalidOperationException: Cache entry must specify a value for Size when SizeLimit is set.
   at Microsoft.Extensions.Caching.Memory.MemoryCache.SetEntry(CacheEntry entry)
   at Microsoft.Extensions.Caching.Memory.CacheEntry.Dispose()

I think “Size = 1” can be added here: https://github.com/stefanprodan/AspNetCoreRateLimit/blob/c5a4f28010d8a7db9c095633573d4b80ce685dbf/src/AspNetCoreRateLimit/Store/MemoryCacheRateLimitStore.cs#L43

[onionhammer]

@cristipufu yes, exactly; a separate MemoryCache (stored in a separate ServiceProvider instance) without depending on the end-developer to set up a memorycache without a SizeLimit.

[mirdaki]

@thomasgalliker I think your solution is a reasonable workaround! I will note that it looks like you’re creating two separate instances of the memory cache, which might cause some issues. I would suggest updating your static class to store the cache ones it’s created, then just return the stored cache instead of creating new ones each time.