This project is archived and is in readonly mode.

Changeset [b53f00690173797a39ff46e55dd25c20581c3d00] by Yehuda Katz

August 3rd, 2009 @ 12:39 AM

Remove legacy processing and content_length

  • convert_content_type! is handled by assign_default_content_type_and_charset!
  • set_content_length! should be handled by the endpoint server. Otherwise each middleware that modifies the body has to do the expensive work of recalculating content_length.
  • convert_language! appears to be legacy. There are no tests for this
  • convert_cookies! should be handled by the new HeaderHash in Rack
  • Use an integer for .status's internal representation to avoid needing to do String manipulation just to find out the status http://github.com/rails/rails/co...

Committed by Yehuda Katz

  • M actionpack/lib/action_dispatch/http/response.rb
  • M actionpack/test/dispatch/request/multipart_params_parsing_test.rb
  • M actionpack/test/dispatch/response_test.rb
  • M actionpack/test/new_base/base_test.rb

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>