aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/http
Commit message (Collapse)AuthorAgeFilesLines
* ActionController::Metal can be a middlewareYehuda Katz + Carl Lerche2009-08-251-1/+4
|
* Got tests to pass with some more changes.Yehuda Katz2009-08-151-24/+12
| | | | | | | | | | | | | | | | * request.formats is much simpler now * For XHRs or Accept headers with a single item, we use the Accept header * For other requests, we use params[:format] or fallback to HTML * This is primarily to work around the fact that browsers provide completely broken Accept headers, so we have to whitelist the few cases we can specifically isolate and treat other requests as coming from the browser * For APIs, we can support single-item Accept headers, which disambiguates from the browsers * Requests to an action that only has an XML template from the browser will no longer find the template. This worked previously because most browsers provide a catch-all */*, but this was mostly accidental behavior. If you want to serve XML, either use the :xml format in links, or explicitly specify the XML template: render "template.xml".
* More perf work:Yehuda Katz2009-08-112-58/+63
| | | | | | | | | | | | | | * Move #set_cookie and #delete_cookie inline to optimize. These optimizations should almost certainly be sent back upstream to Rack. The optimization involves using an ivar for cookies instead of indexing into the headers each time. * Was able to use a bare Hash for headers now that cookies have their own joining semantics (some code assumed that the raw cookies were an Array). * Cache blankness of body on body= * Improve expand_cache_key for Arrays of a single element (common in our case) * Use a simple layout condition check unless conditions are used * Cache visible actions * Lazily load the UrlRewriter * Make etag an ivar that is set on prepare!
* Got overhead down from 127 to 85. All tests pass:Yehuda Katz2009-08-111-36/+56
| | | | | | | | | | | | | | | | | | | | | | | | | * Tentatively replaced HeaderHash with SimpleHeaderHash, which does not preserve case but does handle converting Arrays to Strings in to_hash. This requires further discussion. * Moved default_charset to ActionDispatch::Response to avoid having to hop over to ActionController. Ideally, this would be a constant on AD::Response, but some tests expect to be able to change it dynamically and I didn't want to change them yet. * Completely override #initialize from Rack::Response. Previously, it was creating a HeaderHash, and then we were creating an entirely new one. There is no way to call super without incurring the overhead of creating a HeaderHash. * Override #write from Rack::Response. Its implementation tracks Content-Length, and doing so adds additional overhead that could be mooted if other middleware changes the body. It is more efficiently done at the top-level server. * Change sending_file to an instance_variable instead of header inspection. In general, if a state is important, it should be set as a property of the response not reconstructed later. * Set the Etag to @body instead of .body. AS::Cache.expand_cache_key handles Arrays fine, and it's more efficient to let it handle the body parts, since it is not forced to create a joined String. * If we detect the default cache control case, just set it, rather than setting the constituent parts and then running the normal (expensive) code to generate the string.
* This change causes some failing tests, but it should be possible to make ↵Yehuda Katz2009-08-111-11/+12
| | | | them pass with minimal performance impact.
* Allow to configure trusted proxies via ↵Felipe Talavera2009-08-091-2/+2
| | | | | | ActionController::Base.trusted_proxies [#2126 state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* Remove legacy processing and content_lengthYehuda Katz2009-08-021-40/+10
| | | | | | | | | | | * 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
* Update cache_control to be a Hash of options that is used to build the header.Yehuda Katz2009-08-021-5/+19
| | | | * Significantly simplifies setting and modifying cache control in other areas
* Remove last TODO.José Valim2009-07-291-1/+1
| | | | Signed-off-by: Yehuda Katz <wycats@gmail.com>
* Remove Mime::ALL from Mime::SET.José Valim2009-07-291-2/+4
| | | | Signed-off-by: Yehuda Katz <wycats@gmail.com>
* Refactor even more Responder. Move mime negotiation to request and added ↵José Valim2009-07-291-8/+34
| | | | | | respond_to class method. Signed-off-by: Yehuda Katz <wycats@gmail.com>
* Small changes to get 1.9 passing (for the most part)Yehuda Katz2009-06-231-1/+1
|
* Fixing pending tests and fixed some formats / partial rendering semanticsYehuda Katz + Carl Lerche2009-06-172-7/+17
|
* Finish making things pass with updated internal content_type semanticsYehuda Katz + Carl Lerche2009-06-151-45/+11
|
* Don't want Rack's Content-Type default header thoughJeremy Kemper2009-05-241-2/+3
|
* Construct a single HeaderHash. Speed up assign_default_content_type_and_charsetJeremy Kemper2009-05-241-4/+10
|
* Speed up Request#formatsJeremy Kemper2009-05-231-6/+14
|
* Slightly modify things to get content type matching working without breaking ↵Yehuda Katz2009-05-231-2/+8
| | | | other code
* Remove some response content type concepts from ActionViewYehuda Katz + Carl Lerche2009-05-212-2/+2
|
* Fixed new_base tests on ruby 1.9Yehuda Katz + Carl Lerche2009-05-201-1/+1
|
* Cherry-picking Active Support extensionsJeremy Kemper2009-05-141-2/+4
|
* Cherry-pick core extensionsJeremy Kemper2009-05-133-0/+3
|
* Implemented redirects and partial rendering in new base.Yehuda Katz + Carl Lerche2009-05-121-6/+0
|
* Functional test runner finalizes response just like the integration test ↵Joshua Peek2009-05-021-1/+1
| | | | | runner. In both runners, the @response object will now behave the same. Some functional tests will need to be updated if they are relying on preprocessed data on the response.
* Deprecate assert_redirect_to's partial hash matchingJoshua Peek2009-05-021-2/+0
|
* Move TestRequest#query_parameters into AD TestRequestJoshua Peek2009-04-301-1/+1
|
* Start moving TestRequest and TestResponse into ActionDispatchJoshua Peek2009-04-302-72/+20
|
* Group integration test helpers and delegate other helpers to request and ↵Joshua Peek2009-04-301-0/+9
| | | | response objects
* Switch to action_dispatch rack namespace Joshua Peek2009-04-301-2/+2
|
* Deprecate template, session, assigns, and layout accessors on response ↵Joshua Peek2009-04-281-3/+13
| | | | object. Instead access them through the controller instance. This mainly affects functional test assertions.
* Deprecate response.assignsJoshua Peek2009-04-281-2/+6
|
* Delegate controller.session to request.session and deprecate response session Joshua Peek2009-04-272-9/+5
|
* Array splitting strings on newlines is deprecatedJeremy Kemper2009-04-261-4/+15
|
* Not sure why Request#session is missingJeremy Kemper2009-04-261-0/+8
|
* Check for to_str rather than StringJeremy Kemper2009-04-261-3/+1
|
* Inherit TestSession from Session::AbstractStore and add indifferent access ↵Joshua Peek2009-04-262-12/+5
| | | | to Session::AbstractStore.
* Move useful response test helpers into requestJoshua Peek2009-04-261-2/+63
|
* Remove vendored version of RackJoshua Peek2009-04-251-1/+5
|
* Remove pending rack specifications until they are officialJoshua Peek2009-04-251-2/+2
|
* Use rack's status code list as a baseJoshua Peek2009-04-141-67/+19
|
* We aren't using UploadedStringIO and UploadedTempfile anymoreJoshua Peek2009-04-141-0/+28
|
* Bring abstract_controller up to date with rails/masterCarl Lerche & Yehuda Katz2009-04-132-47/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolved all the conflicts since 2.3.0 -> HEAD. Following is a list of commits that could not be applied cleanly or are obviated with the abstract_controller refactor. They all need to be revisited to ensure that fixes made in 2.3 do not reappear in 3.0: 2259ecf368e6a6715966f69216e3ee86bf1a82a7 AR not available * This will be reimplemented with ActionORM or equivalent 06182ea02e92afad579998aa80144588e8865ac3 implicitly rendering a js response should not use the default layout [#1844 state:resolved] * This will be handled generically 893e9eb99504705419ad6edac14d00e71cef5f12 Improve view rendering performance in development mode and reinstate template recompiling in production [#1909 state:resolved] * We will need to reimplement rails-dev-boost on top of the refactor; the changes here are very implementation specific and cannot be cleanly applied. The following commits are implicated: 199e750d46c04970b5e7684998d09405648ecbd4 3942cb406e1d5db0ac00e03153809cc8dc4cc4db f8ea9f85d4f1e3e6f3b5d895bef6b013aa4b0690 e3b166aab37ddc2fbab030b146eb61713b91bf55 ae9f258e03c9fd5088da12c1c6cd216cc89a01f7 44423126c6f6133a1d9cf1d0832b527e8711d40f 0cb020b4d6d838025859bd60fb8151c8e21b8e84 workaround for picking layouts based on wrong view_paths [#1974 state:resolved] * The specifics of this commit no longer apply. Since it is a two-line commit, we will reimplement this change. 8c5cc66a831aadb159f3daaffa4208064c30af0e make action_controller/layouts pick templates from the current instance's view_paths instead of the class view_paths [#1974 state:resolved] * This does not apply at all. It should be trivial to apply the feature to the reimplemented ActionController::Base. 87e8b162463f13bd50d27398f020769460a770e3 fix HTML fallback for explicit templates [#2052 state:resolved] * There were a number of patches related to this that simply compounded each other. Basically none of them apply cleanly, and the underlying issue needs to be revisited. After discussing the underlying problem with Koz, we will defer these fixes for further discussion.
* Remove errant debug codeYehuda Katz2009-02-031-2/+0
|
* Merge commit 'rails/3-0-unstable'Yehuda Katz2009-02-021-1/+0
|\
| * Remove dead cgi_ext requireJeremy Kemper2009-01-311-1/+0
| |
* | Some merge cleanupYehuda Katz2009-02-021-10/+11
| |
* | Merge commit 'rails/3-0-unstable'Yehuda Katz2009-01-301-22/+9
|/ | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/base.rb actionpack/lib/action_dispatch/http/mime_type.rb actionpack/lib/action_dispatch/http/request.rb actionpack/lib/action_view/base.rb actionpack/lib/action_view/paths.rb actionpack/test/controller/session/cookie_store_test.rb actionpack/test/dispatch/rack_test.rb actionpack/test/dispatch/request_test.rb
* Move HTTP libs and middleware into ActionDispatch componentJoshua Peek2009-01-276-0/+1103