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

Merge pull request #339 from brian-kephart/defined

Browse source 
Replace `const_defined?` with `defined?`
This commit is contained in:
Gonzalo Rodriguez 2018-05-15 10:51:25 -03:00 committed by GitHub
commit 0476778a8a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
lib/rack/attack/store_proxy/redis_store_proxy.rb
View file

@ -5,14 +5,7 @@ module Rack
module StoreProxy module StoreProxy
class RedisStoreProxy < SimpleDelegator class RedisStoreProxy < SimpleDelegator
def self.handle?(store) def self.handle?(store)
# Using const_defined? for now. defined?(::Redis::Store) && store.is_a?(::Redis::Store)
#
# Go back to use defined? once this ruby issue is
# fixed and released:
# https://bugs.ruby-lang.org/issues/14407
#
# defined?(::Redis::Store) && store.is_a?(::Redis::Store)
const_defined?("::Redis::Store") && store.is_a?(::Redis::Store)
end end
def initialize(store) def initialize(store)