Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Exception when seeding policy store with distributed SQL Server cache in 3.0.2 #68

Closed
johansme opened this issue Feb 26, 2019 · 1 comment
Labels

Comments

@johansme
Copy link

The SeedAsync method of the DistributedCacheIpPolicyStore and the DistributedCacheClientPolicyStore both call the SetAsync method of the DistributedCacheRateLimitStore with only the id and entry parameters set.
This means that the IDistributedCache's SetStringAsync method is called with the options (third) parameter set to null.
However, when using a DistributedSqlServerCache as the IDistributedCache, this null argument is propagated to the SetAsync method, where it throws an ArgumentNullException. In other words, seeding the policy store can never succeed when the distributed cache is an SQL Server cache.

@cristipufu cristipufu added the bug label Feb 26, 2019
cristipufu added a commit that referenced this issue Feb 28, 2019
@cristipufu
Copy link
Collaborator

Hi @johansme, thanks for reporting this.

Indeed, it seems there's a null parameters check:

  1. RedisCache
  2. SqlServerCache

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants