diff options
author | Jeremy Evans <code@jeremyevans.net> | 2022-05-10 14:51:47 -0700 |
---|---|---|
committer | Jeremy Evans <code@jeremyevans.net> | 2022-05-10 15:13:05 -0700 |
commit | 297880050290ad04ce5bfd65c87ff5431495dbe1 (patch) | |
tree | d1363ef9cea4de5b34b81b15121215c50a08bf37 | |
parent | 67d2e0d3a2027f6c97166f960e327335c5620339 (diff) | |
download | rack-297880050290ad04ce5bfd65c87ff5431495dbe1.tar.gz |
Remove Rack::Files.method_added
The comment says this should be removed in Rack 3. This was added in Rack 2.2, so it should be safe to remove now.
-rw-r--r-- | lib/rack/files.rb | 8 | ||||
-rw-r--r-- | test/spec_files.rb | 8 |
2 files changed, 0 insertions, 16 deletions
diff --git a/lib/rack/files.rb b/lib/rack/files.rb index d5fba0d6..f0b0dce4 100644 --- a/lib/rack/files.rb +++ b/lib/rack/files.rb @@ -22,14 +22,6 @@ module Rack ALLOW_HEADER = ALLOWED_VERBS.join(', ') MULTIPART_BOUNDARY = 'AaB03x' - # @todo remove in 3.0 - def self.method_added(name) - if name == :response_body - raise "#{self.class}\#response_body is no longer supported." - end - super - end - attr_reader :root def initialize(root, headers = {}, default_mime = 'text/plain') diff --git a/test/spec_files.rb b/test/spec_files.rb index 75b21d8d..f74e5195 100644 --- a/test/spec_files.rb +++ b/test/spec_files.rb @@ -28,14 +28,6 @@ describe Rack::Files do assert_equal 200, app.serving(request, file_path)[0] end - it 'raises if you attempt to define response_body in subclass' do - c = Class.new(Rack::Files) - - lambda do - c.send(:define_method, :response_body){} - end.must_raise RuntimeError - end - it 'serves files with + in the file name' do Dir.mktmpdir do |dir| File.write File.join(dir, "you+me.txt"), "hello world" |