From 0fe30e3a3deb92ce58b7d5087d7f2ce4e3b2c8fc Mon Sep 17 00:00:00 2001 From: Gonzalo Rodriguez Date: Fri, 23 Mar 2018 10:58:47 -0300 Subject: [PATCH] Don't autoload when it's barely valuable to do so Rack::Attack::PathNormalizer and Rack::Attack::Request are both used in #call method, which is going to be used by every rack-attack user as long as they insert the middleware in their app. --- lib/rack/attack.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/rack/attack.rb b/lib/rack/attack.rb index 8c9df48..dd151fd 100644 --- a/lib/rack/attack.rb +++ b/lib/rack/attack.rb @@ -1,12 +1,13 @@ require 'rack' require 'forwardable' +require 'rack/attack/path_normalizer' +require 'rack/attack/request' class Rack::Attack class MisconfiguredStoreError < StandardError; end class MissingStoreError < StandardError; end autoload :Cache, 'rack/attack/cache' - autoload :PathNormalizer, 'rack/attack/path_normalizer' autoload :Check, 'rack/attack/check' autoload :Throttle, 'rack/attack/throttle' autoload :Safelist, 'rack/attack/safelist' @@ -18,7 +19,6 @@ class Rack::Attack autoload :RedisStoreProxy, 'rack/attack/store_proxy/redis_store_proxy' autoload :Fail2Ban, 'rack/attack/fail2ban' autoload :Allow2Ban, 'rack/attack/allow2ban' - autoload :Request, 'rack/attack/request' class << self