Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | edit pass over all warnings | Xavier Noria | 2014-10-28 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | This patch uniformizes warning messages. I used the most common style already present in the code base: * Capitalize the first word. * End the message with a full stop. * "Rails 5" instead of "Rails 5.0". * Backticks for method names and inline code. Also, converted a few long strings into the new heredoc convention. | ||||
* | let's warn with heredocs | Xavier Noria | 2014-10-28 | 1 | -1/+7 |
| | | | | | | | | | | | | The current style for warning messages without newlines uses concatenation of string literals with manual trailing spaces where needed. Heredocs have better readability, and with `squish` we can still produce a single line. This is a similar use case to the one that motivated defining `strip_heredoc`, heredocs are super clean. | ||||
* | remove duplicate method (_status_code) in action_dispatch | Abdelkader Boudih | 2014-10-19 | 1 | -3/+0 |
| | |||||
* | Make _status_code methods nodoc | Prathamesh Sonpatki | 2014-10-19 | 1 | -1/+1 |
| | | | | | - Also one minor change for documenting url_for method in ActionController::Metal. [ci skip] | ||||
* | Add support for Rack::ContentLength middelware | Javan Makhmali | 2014-09-06 | 1 | -0/+4 |
| | |||||
* | Deprecate implicit AD::Response splatting and Array conversion | Jeremy Kemper | 2014-09-06 | 1 | -2/+13 |
| | |||||
* | Handle client disconnect during live streaming | Matthew Draper | 2014-06-08 | 1 | -13/+47 |
| | | | | .. even when the producer is blocked for a write. | ||||
* | fixes stack level too deep exception on action named 'status' returning ↵ | Christiaan Van den Poel | 2014-05-15 | 1 | -0/+3 |
| | | | | 'head :ok' | ||||
* | 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 | ||||
* | Do note remove `Content-Type` when `render :body` | Prem Sichanugrist | 2014-03-05 | 1 | -12/+1 |
| | | | | | | | | | | | | | `render :body` should just not set the `Content-Type` header. By removing the header, it breaks the compatibility with other parts. After this commit, `render :body` will returns `text/html` content type, sets by default from `ActionDispatch::Response`, and it will preserve the overridden content type if you override it. Fixes #14197, #14238 This partially reverts commit 3047376870d4a7adc7ff15c3cb4852e073c8f1da. | ||||
* | Add `#no_content_type` attribute to `AD::Response` | Prem Sichanugrist | 2014-02-18 | 1 | -2/+13 |
| | | | | | Setting this attribute to `true` will remove the content type header from the request. This is use in `render :body` feature. | ||||
* | Introduce `render :body` for render raw content | Prem Sichanugrist | 2014-02-18 | 1 | -1/+1 |
| | | | | | | | | | | | | This is an option for sending a raw content back to browser. Note that this rendering option will unset the default content type and does not include "Content-Type" header back in the response. You should only use this option if you are expecting the "Content-Type" header to not be set. More information on "Content-Type" header can be found on RFC 2616, section 7.2.1. Please see #12374 for more detail. | ||||
* | Prevent [response].flatten from recursing infinitely. | Dan Kang | 2014-02-08 | 1 | -1/+1 |
| | | | | | | Returning `self` from within the array returned by `to_ary` caused this. Instead, we can just substitute another object. It provides the `each` behavior required by the rack spec. | ||||
* | FilterRedirect is referenced at the class level from the Response | Aaron Patterson | 2014-01-31 | 1 | -0/+1 |
| | | | | | We can just require the file rather than going through the autoload indirection | ||||
* | Remove deprecated cattr_* requires | Genadi Samokovarov | 2013-12-03 | 1 | -1/+1 |
| | |||||
* | Reuse variable to avoid symbol usage | Carlos Antonio da Silva | 2013-08-31 | 1 | -2/+2 |
| | |||||
* | update Rails::Railtie::Configuration and ↵ | Jon Kessler | 2013-08-16 | 1 | -1/+1 |
| | | | | ActionDispatch::Response#respond_to? to accept include_private argument | ||||
* | writing the new body can cause the response to be committed and the | Aaron Patterson | 2013-06-28 | 1 | -1/+3 |
| | | | | | | | | request thread to return up the stack before the instance variable is assigned. Synchronize so that the ivar is assigned before the other thread can activate. fixes #10984 | ||||
* | Add propery docs to ActionDispatch::Response [ci skip] | Steve Klabnik | 2013-05-30 | 1 | -3/+19 |
| | | | | | | | | | | | | | After some discussion on Twitter with @skud, the documentation on ActionDispatch::Response is a bit sparse. This class is useful when you're writing tests, as often you want to assert various things about the response that's coming back. Better docs would make this easier for people new to testing in Rails. I only added some descriptions for various properties that were defined, and mostly just a sentence or two. Most of these things are familliar if you're working with HTTP, but some words is better than no words at all. Hopefully further commits will fix up things that aren't just documentation. | ||||
* | extract no content response codes to a constant | Vipul A M | 2013-04-22 | 1 | -1/+2 |
| | |||||
* | Removed an old require of an MD5 digest which is no longer used. | wangjohn | 2013-02-15 | 1 | -1/+0 |
| | |||||
* | Avoid to_sym calls | Santiago Pastorino | 2013-02-13 | 1 | -1/+1 |
| | |||||
* | generic pass before merging docrails | Xavier Noria | 2013-01-26 | 1 | -5/+7 |
| | |||||
* | Add documentation to ActionDispatch::Response | Matthew Stopa | 2013-01-20 | 1 | -0/+7 |
| | |||||
* | Fix typo introduced in 0004ca3a | Carlos Antonio da Silva | 2013-01-16 | 1 | -1/+1 |
| | | | | [ci skip] | ||||
* | More documentation for ActionDispatch::Response | Matthew Stopa | 2013-01-16 | 1 | -1/+3 |
| | | | | [ci skip] | ||||
* | Document ActionDispatch::Response#body method | Matthew Stopa | 2013-01-16 | 1 | -0/+1 |
| | |||||
* | charset should not be appended for `head` responses | Yves Senn | 2012-12-31 | 1 | -2/+6 |
| | | | | | | | 1) Failure: test_head_created_with_image_png_content_type(RenderTest) [test/controller/render_test.rb:1238]: Expected: "image/png" Actual: "image/png; charset=utf-8" | ||||
* | Adding filter capability to ActionController logs | Fabrizio Regini | 2012-12-05 | 1 | -0/+1 |
| | |||||
* | push header merge down to a private method so that live responses can have ↵ | Aaron Patterson | 2012-08-13 | 1 | -3/+7 |
| | | | | their own header object | ||||
* | introduce default_headers config | Egor Homakov | 2012-08-09 | 1 | -0/+5 |
| | |||||
* | load active_support/core_ext/module/delegation in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | header hash is duped before being sent up the rack stack | Aaron Patterson | 2012-07-29 | 1 | -12/+16 |
| | |||||
* | ActionController::DataStreaming::FileBody doesn't respond to #close. | Rafael Mendonça França | 2012-07-29 | 1 | -1/+1 |
| | |||||
* | flushing output should write to the stream rather than mutating the response ↵ | Aaron Patterson | 2012-07-29 | 1 | -8/+12 |
| | | | | object | ||||
* | adding a buffered stream to the response object | Aaron Patterson | 2012-07-29 | 1 | -2/+42 |
| | |||||
* | initialize ivars, refactor recycle! to call initialize | Aaron Patterson | 2012-07-29 | 1 | -0/+2 |
| | |||||
* | threads can wait on responses to be committed | Aaron Patterson | 2012-07-27 | 1 | -1/+24 |
| | |||||
* | Fix comment in AD http response | Alexey Vakhov | 2012-04-13 | 1 | -1/+1 |
| | | | | response in integration test accessible without @ via attr_reader | ||||
* | Add ActionController::HTTP | Santiago Pastorino | 2012-03-14 | 1 | -2/+7 |
| | | | | | | More info http://edgeguides.rubyonrails.org/api_app.html [Carlos Antonio da Silva & Santiago Pastorino] | ||||
* | remove check for string from request body setter | lest | 2011-12-21 | 1 | -8/+1 |
| | | | | it was required on ruby 1.8 | ||||
* | use Array#join so that file encoding doesn't impact returned string. | Aaron Patterson | 2011-12-12 | 1 | -3/+3 |
| | | | | Fixes #3957 | ||||
* | Use freezed string constant. Some string literals is used repeatedly. | kennyj | 2011-12-08 | 1 | -8/+10 |
| | |||||
* | Use default charset when we read content type without charset. | kennyj | 2011-12-07 | 1 | -1/+1 |
| | |||||
* | removed deprecated methods, and related tests, from ActionPack | Josh Kalderimis | 2011-05-24 | 1 | -23/+0 |
| | |||||
* | Make :status an attr_reader to avoid unnecessary warning when replacing status= | wycats | 2011-05-22 | 1 | -1/+2 |
| | |||||
* | Fix previous commit by allowing a proc to be given as response_body. This is ↵ | José Valim | 2011-05-10 | 1 | -15/+41 |
| | | | | deprecated and is going to be removed in future releases. | ||||
* | Fix tests on 1.8 by explicitly checking for strings (which also improves ↵ | José Valim | 2011-04-20 | 1 | -1/+9 |
| | | | | performance). |