| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Remove unnecessary require pathname from actionpack controller specs | Abhay Nikam | 2019-06-02 | 1 | -1/+0 |
| | | |||||
| * | Change `ActionDispatch::Response#content_type` returning Content-Type header ↵ | yuuji.yaginuma | 2019-06-01 | 1 | -7/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | as it is Since #35709, `Response#conten_type` returns only MIME type correctly. It is a documented behavior that this method only returns MIME type, so this change seems appropriate. https://github.com/rails/rails/blob/39de7fac0507070e3c5f8b33fbad6fced84d97ed/actionpack/lib/action_dispatch/http/response.rb#L245-L249 But unfortunately, some users expect this method to return all Content-Type that does not contain charset. This seems to be breaking changes. We can change this behavior with the deprecate cycle. But, in that case, a method needs that include Content-Type with additional parameters. And that method name is probably the `content_type` seems to properly. So I changed the new behavior to more appropriate `media_type` method. And `Response#content_type` changed (as the method name) to return Content-Type header as it is. Fixes #35709. [Rafael Mendonça França & Yuuji Yaginuma ] | ||||
| * | Use frozen string literal in actionpack/ | Kir Shatrov | 2017-07-29 | 1 | -0/+2 |
| | | |||||
| * | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 1 | -1/+0 |
| | | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
| * | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 1 | -0/+1 |
| | | |||||
| * | Add more rubocop rules about whitespaces | Rafael Mendonça França | 2016-10-29 | 1 | -1/+1 |
| | | |||||
| * | Add three new rubocop rules | Rafael Mendonça França | 2016-08-16 | 1 | -1/+1 |
| | | | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository. | ||||
| * | Add `Style/EmptyLines` in `.rubocop.yml` and remove extra empty lines | Ryuta Kamizono | 2016-08-07 | 1 | -1/+0 |
| | | |||||
| * | modernizes hash syntax in actionpack | Xavier Noria | 2016-08-06 | 1 | -11/+11 |
| | | |||||
| * | applies new string literal convention in actionpack/test | Xavier Noria | 2016-08-06 | 1 | -21/+21 |
| | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
| * | Stop using deprecated `render :text` in test | Prem Sichanugrist | 2015-07-17 | 1 | -1/+1 |
| | | | | | | | | | | This will silence deprecation warnings. Most of the test can be changed from `render :text` to render `:plain` or `render :body` right away. However, there are some tests that needed to be fixed by hand as they actually assert the default Content-Type returned from `render :body`. | ||||
| * | Migrating xhr methods to keyword arguments syntax | Kir Shatrov | 2015-02-01 | 1 | -2/+2 |
| | | | | | | | | | | | | | in `ActionController::TestCase` and `ActionDispatch::Integration` Old syntax: `xhr :get, :create, params: { id: 1 }` New syntax example: `get :create, params: { id: 1 }, xhr: true` | ||||
| * | Address CVE-2014-4671 (JSONP Flash exploit) | Greg Campbell | 2014-07-09 | 1 | -1/+1 |
| | | | | | | | Adds a comment before JSONP callbacks. See http://miki.it/blog/2014/7/8/abusing-jsonp-with-rosetta-flash/ for more details on the exploit in question. | ||||
| * | CSRF protection from cross-origin <script> tags | Jeremy Kemper | 2013-12-17 | 1 | -2/+2 |
| | | | | | Thanks to @homakov for sounding the alarm about JSONP-style data leaking | ||||
| * | Add changelog entry for jsonp mimetype change, fix failing test | Carlos Antonio da Silva | 2012-04-29 | 1 | -1/+1 |
| | | | | | Fix failing test: Mime::JS generates "text/javascript" | ||||
| * | Merge pull request #2321 from omjokine/master | José Valim | 2012-04-29 | 1 | -1/+1 |
| |\ | | | | | JSONP should use mimetype application/javascript | ||||
| | * | fixed test case test_render_json_with_callback to use content_type ↵ | Olli Jokinen | 2011-07-28 | 1 | -1/+1 |
| | | | | | | | | | application/javascript | ||||
| * | | Moving to logger methods to ActiveSupport Logger | Karunakar (Ruby) | 2011-12-24 | 1 | -1/+2 |
| | | | |||||
| * | | Revert the serializers API as other alternatives are now also under discussion | José Valim | 2011-11-25 | 1 | -48/+0 |
| | | | |||||
| * | | Rely solely on active_model_serializer and remove the fancy constant lookup. | José Valim | 2011-11-23 | 1 | -1/+15 |
| | | | |||||
| * | | Initial commit of serializer support | Jose and Yehuda | 2011-10-15 | 1 | -0/+34 |
| |/ | |||||
| * | render_to_string must ensure that response_body | Neeraj Singh | 2011-01-25 | 1 | -0/+10 |
| | | | | | | | | | is nil [ #5875 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | renderer calls object.to_json when rendering :json => object [#5655 ↵ | Diego Carrion | 2010-09-27 | 1 | -0/+13 |
| | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Always downstream given options in :json, :xml and :js renderers and add ↵ | José Valim | 2010-04-22 | 1 | -0/+18 |
| | | | | | tests for it. | ||||
| * | Cleanup render callstack and make render(:json => {}, :status => 401) work ↵ | José Valim | 2010-02-24 | 1 | -0/+10 |
| | | | | | again. | ||||
| * | Namespace TestControllers inside their test case class | Joshua Peek | 2009-09-19 | 1 | -27/+31 |
| | | |||||
| * | Ported over render :file tests. | Yehuda Katz + Carl Lerche | 2009-05-13 | 1 | -0/+80 |
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |