This project is archived and is in readonly mode.
[PATCH] re-enable send_file(path, :stream => true)
Reported by Kouhei Sutou | October 8th, 2009 @ 09:54 AM | in 3.0.2
3.0.pre disables send_file(path, :stream => true)
temporary.
There is a comment in
actionpack/lib/action_controller/metal/streaming.rb:
# TODO : Make render :text => proc {} work with the new base
The attached patch re-supports :text => proc {} and re-enables :stream => true option.
Comments and changes to this ticket
-
Kouhei Sutou October 8th, 2009 @ 09:55 AM
- no changes were found...
-
Kouhei Sutou October 9th, 2009 @ 01:04 AM
- Title changed from re-enable send_file(path, :stream => true) to [PATCH] re-enable send_file(path, :stream => true)
-
José Valim February 23rd, 2010 @ 07:26 AM
- Assigned user set to Yehuda Katz (wycats)
- Milestone cleared.
-
Yehuda Katz (wycats) February 24th, 2010 @ 05:41 AM
- State changed from new to resolved
This is fixed in master by using the Rack::Sendfile middleware. http://github.com/rails/rails/commit/5e2bd08023344f3fd4675e80203a10...
-
Jeremy Kemper October 15th, 2010 @ 11:01 PM
- Milestone set to 3.0.2
- Importance changed from to High
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>