Do not auto-plug for rails < 5

Author: fatkodima

README.md

@@ -68,7 +68,14 @@ Or install it yourself as:
 
 Then tell your ruby web application to use rack-attack as a middleware.
 
-a) For __rails__ applications it is used by default. You can disable it permanently (like for specific environment) or temporarily (can be useful for specific test cases) by writing:
+a) For __rails__ applications with versions >= 5 it is used by default. For older rails versions you should enable it explicitly:
+```ruby
+# In config/application.rb
+
+config.middleware.use Rack::Attack
+```
+
+You can disable it permanently (like for specific environment) or temporarily (can be useful for specific test cases) by writing:
 
 ```ruby
 Rack::Attack.enabled = false

lib/rack/attack/railtie.rb

@@ -4,21 +4,17 @@ module Rack
   class Attack
     class Railtie < ::Rails::Railtie
       initializer 'rack.attack.middleware', after: :load_config_initializers, before: :build_middleware_stack do |app|
-        middlewares = app.config.middleware
+        if Gem::Version.new(::Rails::VERSION::STRING) >= Gem::Version.new("5")
+          middlewares = app.config.middleware
+          operations = middlewares.send(:operations) + middlewares.send(:delete_operations)
 
-        operations =
-          if Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("5")
-            middlewares.send(:operations) + middlewares.send(:delete_operations)
-          else
-            middlewares.instance_variable_get(:@operations)
+          use_middleware = operations.none? do |operation|
+            middleware = operation[1]
+            middleware.include?(Rack::Attack)
           end
 
-        use_middleware = operations.none? do |operation|
-          middleware = operation[1]
-          middleware.include?(Rack::Attack)
+          middlewares.use(Rack::Attack) if use_middleware
         end
-
-        middlewares.use(Rack::Attack) if use_middleware
       end
     end
   end

spec/acceptance/rails_middleware_spec.rb

@@ -2,7 +2,7 @@
 
 require_relative "../spec_helper"
 
-if defined?(Rails) && Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("5")
+if defined?(Rails)
   describe "Middleware for Rails" do
     before do
       @app = Class.new(Rails::Application) do
@@ -12,21 +12,30 @@
       end
     end
 
-    it "is enabled by default" do
-      @app.initialize!
-      assert_equal 1, @app.middleware.count(Rack::Attack)
-    end
+    if Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("5")
+      it "is used by default" do
+        @app.initialize!
+        assert_equal 1, @app.middleware.count(Rack::Attack)
+      end
+
+      it "is not added when it was added explicitly" do
+        @app.config.middleware.use(Rack::Attack)
+        @app.initialize!
+        assert_equal 1, @app.middleware.count(Rack::Attack)
+      end
 
-    it "is not added when it was added explicitly" do
-      @app.config.middleware.use(Rack::Attack)
-      @app.initialize!
-      assert_equal 1, @app.middleware.count(Rack::Attack)
+      it "is not added when it was explicitly deleted" do
+        @app.config.middleware.delete(Rack::Attack)
+        @app.initialize!
+        refute @app.middleware.include?(Rack::Attack)
+      end
     end
 
-    it "is not added when it was explicitly deleted" do
-      @app.config.middleware.delete(Rack::Attack)
-      @app.initialize!
-      refute @app.middleware.include?(Rack::Attack)
+    if Gem::Version.new(Rails::VERSION::STRING) < Gem::Version.new("5")
+      it "is not used by default" do
+        @app.initialize!
+        assert_equal 0, @app.middleware.count(Rack::Attack)
+      end
     end
   end
 end

spec/rack_attack_spec.rb

@@ -79,21 +79,21 @@
 
   describe 'enabled' do
     it 'should be enabled by default' do
-      Rack::Attack.enabled.must_equal true
+      _(Rack::Attack.enabled).must_equal true
     end
 
     it 'should directly pass request when disabled' do
       bad_ip = '1.2.3.4'
       Rack::Attack.blocklist("ip #{bad_ip}") { |req| req.ip == bad_ip }
 
       get '/', {}, 'REMOTE_ADDR' => bad_ip
-      last_response.status.must_equal 403
+      _(last_response.status).must_equal 403
 
       prev_enabled = Rack::Attack.enabled
       begin
         Rack::Attack.enabled = false
         get '/', {}, 'REMOTE_ADDR' => bad_ip
-        last_response.status.must_equal 200
+        _(last_response.status).must_equal 200
       ensure
         Rack::Attack.enabled = prev_enabled
       end