data-govt-nz / ckanext-security

A CKAN extension to hold various security improvements for CKAN
GNU Affero General Public License v3.0
25 stars 32 forks source link

Fix/move from memcache to redis #16

Closed anotheredward closed 6 years ago

anotheredward commented 6 years ago

To make this PR more appropriate to merge into ckan-core and remove an unecessary dependency we've moved the caching mechanism from memcache to redis for the following features:

There is a concern in that when Memcache fills up, it will delete the least recently used item to make space for a new item, and that Memcache is strict about the amount of space it takes up where as Redis's eviction and maxmemory settings depend on its configuration: https://redislabs.com/redis-enterprise-documentation/faqs/what-happens-when-my-database-fills-up/

TODO: