Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | scheme should contain one or more characters | Aaron Patterson | 2014-06-11 | 1 | -1/+1 | |
| | | ||||||
* | | pull the port out of the options hash once | Aaron Patterson | 2014-06-11 | 1 | -2/+3 | |
| | | ||||||
* | | remove useless nil check | Aaron Patterson | 2014-06-11 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | irb(main):004:0> /foo/ !~ nil => true irb(main):005:0> /foo/ !~ 'bar' => true irb(main):006:0> /foo/ !~ 'foo' => false | |||||
* | | these methods are always called with a tld_parameter | Aaron Patterson | 2014-06-11 | 1 | -3/+3 | |
| | | | | | | | | | | remove the default parameter since the methods are always called with a parameter | |||||
* | | rm `same_host?`. The same conditional is two lines down. | Aaron Patterson | 2014-06-11 | 1 | -5/+3 | |
| | | ||||||
* | | Revert "rm `same_host?`. The same conditional is two lines down." | Aaron Patterson | 2014-06-11 | 1 | -5/+8 | |
| | | | | | | | | This reverts commit 79469b4b0c05a50e19699bc9b568042add2d4987. | |||||
* | | rm `same_host?`. The same conditional is two lines down. | Aaron Patterson | 2014-06-11 | 1 | -8/+5 | |
| | | ||||||
* | | cache host on the stack so we only look it up once | Aaron Patterson | 2014-06-11 | 1 | -8/+9 | |
| | | ||||||
* | | only pull :protocol from the options hash once | Aaron Patterson | 2014-06-11 | 1 | -2/+3 | |
| | | ||||||
* | | cache protocol on the stack to reduce options hash lookups | Aaron Patterson | 2014-06-11 | 1 | -7/+7 | |
| | | ||||||
* | | eliminate nil checks in normalize_port | Aaron Patterson | 2014-06-11 | 1 | -7/+7 | |
| | | ||||||
* | | reduce hash lookups and disconnect normalize_port from the options hash | Aaron Patterson | 2014-06-11 | 1 | -9/+9 | |
| | | ||||||
* | | Handle client disconnect during live streaming | Matthew Draper | 2014-06-08 | 1 | -13/+47 | |
| | | | | | | | | .. even when the producer is blocked for a write. | |||||
* | | Use a frozen Set instance for CGI_VARIABLES. | Nathaniel Bibler | 2014-06-05 | 1 | -8/+21 | |
| | | | | | | Also expand the CGI_VARIABLE name listing to multiple lines for cleaner diffs and legibility. | |||||
* | | Merge pull request #15349 from tgxworld/remove_duplicated_method_call | Rafael Mendonça França | 2014-06-03 | 2 | -3/+2 | |
|\ \ | | | | | | | Remove duplicated HashWithIndifferentAccess#with_indifferent_access. | |||||
| * | | Remove duplicated HashWithIndifferentAccess#with_indifferent_access. | Guo Xiang Tan | 2014-05-26 | 2 | -3/+2 | |
| | | | ||||||
* | | | rm dead code | Aaron Patterson | 2014-05-27 | 1 | -5/+0 | |
| | | | ||||||
* | | | PARAMETERS_KEY is only used in the request, so move the constant there | Aaron Patterson | 2014-05-27 | 1 | -2/+4 | |
| | | | ||||||
* | | | rm reset_parameters because we automatically do it from 9ca4839a | Aaron Patterson | 2014-05-27 | 1 | -4/+0 | |
| | | | ||||||
* | | | move path_parameter encoding check to the request object | Aaron Patterson | 2014-05-27 | 1 | -0/+11 | |
|/ / | ||||||
* | | Merge pull request #14986 from dlangevin/trailing-slash-url-generation | Rafael Mendonça França | 2014-05-24 | 1 | -7/+13 | |
|\ \ | | | | | | | | | | | | | | | | | | | Fixes URL generation with trailing_slash: true Conflicts: actionpack/lib/action_dispatch/http/url.rb | |||||
| * | | Fixes URL generation with trailing_slash: true | Dan Langevin | 2014-05-22 | 1 | -9/+17 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | URL generation with trailing_slash: true was adding a trailing slash after .:format Routes.draw do resources :bars end bars_url(trailing_slash: true, format: 'json') # => /bars.json/ This commit removes that extra trailing slash | |||||
* | | | just return path parameters | Aaron Patterson | 2014-05-22 | 1 | -2/+1 | |
| | | | ||||||
* | | | Fix the hash key being delete from env | Rafael Mendonça França | 2014-05-22 | 1 | -1/+1 | |
| | | | | | | | | | | | | It was changed by mistake at c5d64b2b86aa42f57881091491ee289b3c489c7e. | |||||
* | | | use the existing constant rather than the hardcoded string | Aaron Patterson | 2014-05-21 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | so tht I don't go totally insane with THsi crazy hash driven development. why is Everything a hash? Why do people think hashes in hashes with random keys is a Good API? You can't find things or deprecate them or control access whatsoever, you just have to hope that everyone is like "oh, you want to change that? that's cool! we all know it's hashes so go for it!" The End. | |||||
* | | | fewer string allocations per url_for | Aaron Patterson | 2014-05-20 | 1 | -7/+5 | |
| | | | ||||||
* | | | mutate the path string to avoid object allocations | Aaron Patterson | 2014-05-20 | 1 | -4/+4 | |
| | | | ||||||
* | | | push only_path conditional up | Aaron Patterson | 2014-05-20 | 1 | -18/+19 | |
| | | | ||||||
* | | | push arg checking up | Aaron Patterson | 2014-05-20 | 1 | -4/+4 | |
| | | | ||||||
* | | | fewer method calls and string garbage when there is no user/password | Aaron Patterson | 2014-05-20 | 1 | -9/+5 | |
| |/ |/| | ||||||
* | | fixes stack level too deep exception on action named 'status' returning ↵ | Christiaan Van den Poel | 2014-05-15 | 1 | -0/+3 | |
| | | | | | | | | 'head :ok' | |||||
* | | Add RFC4791 MKCALENDAR method | kasper | 2014-05-15 | 1 | -1/+3 | |
| | | ||||||
* | | Always use the provided port for protocol relative urls | Andrew White | 2014-05-11 | 1 | -0/+2 | |
| | | | | | | | | | | There may be situations where you need to tunnel SSL connections over port 80 so we shouldn't remove it if it has been explicitly provided. | |||||
* | | ActionDispatch::Http::URL.normalize_port should not strip port for protocol ↵ | Guilherme Cavalcanti | 2014-05-11 | 1 | -2/+0 | |
| | | | | | | | | relative URL. | |||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2014-05-10 | 1 | -1/+22 | |
|\ \ | ||||||
| * | | copy edits [ci skip] | Vijay Dev | 2014-05-10 | 1 | -8/+8 | |
| | | | ||||||
| * | | [ci skip] doc Http::Headers methods | schneems | 2014-05-09 | 1 | -1/+17 | |
| | | | ||||||
| * | | copy edits [ci skip] | Vijay Dev | 2014-05-08 | 1 | -2/+1 | |
| | | | ||||||
| * | | [ci skip] document ActionDispatch::HTTP::Headers | schneems | 2014-05-07 | 1 | -0/+6 | |
| |/ | ||||||
* | | skip dealing with params if none are provided | Aaron Patterson | 2014-05-09 | 1 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | This lets us avoid 1. A slow call to Hash#slice 2. An is_a? test 3. Extra hash allocations (from slice) 4. String allocations etc. | |||||
* | | don't mutate the options hash, so we don't have to dup | Aaron Patterson | 2014-05-09 | 1 | -4/+3 | |
| | | | | | | | | avoids extra hash allocations on each call | |||||
* | | use unless and || since these options are boolean | Aaron Patterson | 2014-05-09 | 1 | -1/+1 | |
| | | ||||||
* | | HTTP::Headers#key? correctly converts | schneems | 2014-05-07 | 1 | -1/+3 | |
|/ | | | | | | | | | | | | | | | Previously if you were looking for a given key, the header may incorrectly tell you that it did not exist even though it would return a valid value: ```ruby env = { "CONTENT_TYPE" => "text/plain" } headers = ActionDispatch::Http::Headers.new(env) headers["Content-Type"] # => "text/plain" headers.key?("Content-Type") # => false ``` This PR fixes that behavior by converting the key before checking for presence | |||||
* | "subhash" --> "sub-hash" | Akshay Vishnoi | 2014-04-19 | 1 | -2/+2 | |
| | ||||||
* | Implement to_io as an alias | Rafael Mendonça França | 2014-04-17 | 1 | -5/+1 | |
| | ||||||
* | Provide interface for accessing underlying IO object | Tim Linquist | 2014-04-15 | 1 | -0/+5 | |
| | | | | | | In some cases users may need to work with/manipulate more of the Tempfile api than provided by Upload. Allow users to get at the underlying io via the common to_io method of IO/IO-like objects | |||||
* | Return null type format when format is not know | Rafael Mendonça França | 2014-04-14 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | When requesting a controller with the following code with a unknown format: def my_action respond_to do |format| format.json { head :ok } format.any { render text: 'Default response' } end end we should render the default response instead of raising ActionController::UnknownFormat Fixes #14462 Conflicts: actionpack/CHANGELOG.md actionpack/test/controller/mime/respond_with_test.rb Conflicts: actionpack/CHANGELOG.md | |||||
* | Replace trivial regexp with string or index, twice as fast | Kelley Reynolds | 2014-03-28 | 1 | -1/+1 | |
| | ||||||
* | use the body proxy to freeze headers | Aaron Patterson | 2014-03-12 | 1 | -5/+32 | |
| | | | | | | avoid freezing the headers until the web server has actually read data from the body proxy. Once the webserver has read data, then we should throw an error if someone tries to set a header | |||||
* | only write the jar if the response isn't committed | Aaron Patterson | 2014-03-12 | 1 | -0/+4 | |
| | | | | | | | | | | | when streaming responses, we need to make sure the cookie jar is written to the headers before returning up the stack. This commit introduces a new method on the response object that writes the cookie jar to the headers as the response is committed. The middleware and test framework will not write the cookie headers if the response has already been committed. fixes #14352 |