sjs/rack-attack
1
0
Fork 0
mirror of https://github.com/samsonjs/rack-attack.git synced 2026-04-27 15:07:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixes warning instead of hiding it

Browse source 
Warning was: 'warning: instance variable @data not initialized'
This commit is contained in:
Gonzalo Rodriguez 2018-01-23 16:54:13 -03:00
parent e8102910bf
commit 218a320a3d
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Rakefile
View file

@ -10,7 +10,6 @@ namespace :test do
Rake::TestTask.new(:integration) do |t| Rake::TestTask.new(:integration) do |t|
t.pattern = "spec/integration/*_spec.rb" t.pattern = "spec/integration/*_spec.rb"
t.warning = false
end end
end end

3
lib/rack/attack/store_proxy.rb
View file

@ -20,7 +20,10 @@ module Rack
# We also want to use the underlying Dalli client instead of ::ActiveSupport::Cache::MemCacheStore, # We also want to use the underlying Dalli client instead of ::ActiveSupport::Cache::MemCacheStore,
# and the MemCache client if using Rails 3.x # and the MemCache client if using Rails 3.x
if store.instance_variable_defined?(:@data)
client = store.instance_variable_get(:@data) client = store.instance_variable_get(:@data)
end
if ACTIVE_SUPPORT_WRAPPER_CLASSES.include?(store.class.to_s) && ACTIVE_SUPPORT_CLIENTS.include?(client.class.to_s) if ACTIVE_SUPPORT_WRAPPER_CLASSES.include?(store.class.to_s) && ACTIVE_SUPPORT_CLIENTS.include?(client.class.to_s)
client client
else else