-
-
Notifications
You must be signed in to change notification settings - Fork 116
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
ActiveSupport::Cache::MemCacheStore unsupported #50
Comments
Hi Owen! Most people are using rack-throttle for legacy versions of ruby, but if you are able to: https://github.com/kickstarter/rack-attack is much more feature full & maintained! |
Thanks FreekingDean. I'll work on migrating with our next sprint and add any gotchas/comments to this issue. |
Would love to know if there are large sweeping differences, but I believe it was originally based on this repo. |
The switch took about an hour.
The hardest part was understanding how to log if something was throttled. Here was my solution...
Will throw it into production in two weeks and see how it performs. |
Works great. Only issue was that the IP method needs to be overwritten if you deal with HTTP_X_FORWARDED_FOR otherwise you'll block all traffic from your load balancers.
|
Hi.
We recently upgraded from the dalli gem to mem_cache_store. This silently broke the rack-throttle gem since it expects the cache to support
:has_key?
,:get
, and:set
.To fix it, we applied this monkey patch...
I'm not sure if there's a better way but sharing this tidbit for others who may be running this with dalli and then switched to mem_cache_store.
Thanks for this gem!
Owen
The text was updated successfully, but these errors were encountered: