-
Notifications
You must be signed in to change notification settings - Fork 397
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request] Extensibility allows customers to bring their distributed cache or hybrid cache in IdentityModel 8 #2537
Comments
Today, IdentityModel uses the EventBaseLRUCache for two purposes:
This cache is used in the The proposal is to:
We won't touch the InMemoryCryptoProviderCache?
|
@jmprieur Hybrid Cache seems to still be in an API design phase. Would leveraging IDistributedCache on BaseConfiguration be sufficient? I don't think any changes to InMemoryCryptoProviderCache should be part of this feature request. Adding IDistributedCache to IdentityModel.Tokens will set a precedent of adding M.Extensions* to IdentityModel. |
Yes, IDistributedCache would be a wiser choice for the moment. No indeed. we should not touch InMemoryCryptoProviderCache (as I believe I wrote above). Only the BaseConfigurationManager. Yes, let's standardize to v8..0 and v9.0 for net9.0, until net9,.0 GAes at which point we'll move to v.9.0/ That's what the .NET team recommended. |
Hi, I don't know about the specifics of IdentityModel Extensions for .NET, but if there's a caching abstraction to implement (maybe an interface?), then FusionCache can be used as a possible implementation: it is an hybrid cache (L1 or L1+L2), production ready, used for years, fully observable, with resiliency features and more. Here's a nice intro thanks to the hospitality of the On .NET team. Also, Data API Builder uses it as their caching engine. |
Is your feature request related to a problem? Please describe.
Some partners request to have an L1/L2 cache for metadata
Describe the solution you'd like
An extensibility mechanism to provide a distributed cache? IDistributedCache in .NET 9 or the new hybrid cache in .NET 9?
The text was updated successfully, but these errors were encountered: