Removing sendfile middleware from default. Loading sendfile when x_se…

…ndfile_header is true
arunagw · Jul 26, 2011 · f49ca63 · f49ca63
1 parent 36ab1cd
commit f49ca63
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/railties/test/application/middleware_test.rb b/railties/test/application/middleware_test.rb
@@ -31,7 +31,6 @@ def app
         "Rails::Rack::Logger", # must come after Rack::MethodOverride to properly log overridden methods
         "ActionDispatch::ShowExceptions",
         "ActionDispatch::RemoteIp",
-        "Rack::Sendfile",
         "ActionDispatch::Reloader",
         "ActionDispatch::Callbacks",
         "ActiveRecord::ConnectionAdapters::ConnectionManagement",
@@ -55,6 +54,12 @@ def app
       assert_equal "Rack::Cache", middleware.first
     end
 
+    test "Rack::Sendfile is present when action_dispatch.x_sendfile_header is set" do
+      add_to_config "config.action_dispatch.x_sendfile_header = true"
+      boot!
+      assert middleware.include?("Rack::Sendfile")
+    end
+
     test "Rack::SSL is present when force_ssl is set" do
       add_to_config "config.force_ssl = true"
       boot!