Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Use Request#raw_post instead Request#body | Paul Nikitochkin | 2013-07-08 | 1 | -0/+7 |
| | | | | | | | | In order to get raw_post to be not empty after ParamsParser#parse_formatted_parameters, added rewinding of body stream input on parsing json params. Closes #11345 | ||||
* | Merge pull request #11313 from gsamokovarov/url-encoded-params-test | Rafael Mendonça França | 2013-07-06 | 1 | -25/+27 |
|\ | | | | | Cleanup ul_encoded_params_parsing_test | ||||
| * | Cleanup ul_encoded_params_parsing_test | Genadi Samokovarov | 2013-07-05 | 1 | -25/+27 |
| | | |||||
* | | Space is not required for Set-Cookie header | Yamagishi Kazutoshi | 2013-07-05 | 1 | -0/+29 |
|/ | |||||
* | Remove `page_cache_extension` deprecated method | Francesco Rodriguez | 2013-07-01 | 1 | -15/+0 |
| | |||||
* | use bytesize rather than force encoding | Aaron Patterson | 2013-06-28 | 1 | -2/+1 |
| | |||||
* | Re-remove deprecated API. | kennyj | 2013-06-28 | 1 | -7/+0 |
| | |||||
* | Merge pull request #11000 from sbeckeriv/accept_header | Carlos Antonio da Silva | 2013-06-25 | 1 | -0/+6 |
|\ | | | | | | | Fix undefined method `ref' for nil:NilClass for bad accept headers | ||||
| * | Fix undefined method `ref' for nil:NilClass for bad accept headers | Stephen Becker IV | 2013-06-19 | 1 | -0/+6 |
| | | |||||
* | | test-case => failing | Yves Senn | 2013-06-25 | 1 | -0/+13 |
| | | |||||
* | | Merge pull request #11069 from ykzts/actiondispatch-ssl-secure-flag-igonore-case | Guillermo Iguaran | 2013-06-24 | 1 | -0/+14 |
|\ \ | | | | | | | Flag cookies as secure with ignore case in ActionDispatch::SSL | ||||
| * | | Flag cookies as secure with ignore case in ActionDispatch::SSL | Yamagishi Kazutoshi | 2013-06-24 | 1 | -0/+14 |
| | | | |||||
* | | | ActionDispatch:SSL: don't include STS header in non-https responses | Geoff Buesing | 2013-06-23 | 1 | -0/+5 |
|/ / | |||||
* | | Don't remove trailing slash from PATH_INFO for mounted apps | Piotr Sarnacki | 2013-06-21 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when app was mounted as following: class Foo def call(env) [200, {}, [env['PATH_INFO']]] end end RackMountRailsBug::Application.routes.draw do mount RackTest.new => "/foo" end trailing slash was removed from PATH_INFO. For example requesting GET /foo/bar/ on routes defined above would result in a response containing "/foo/bar" instead of "/foo/bar/". This commit fixes the issue. (closes #3215) | ||||
* | | Remove digestor fixtures from AP | Łukasz Strzałkowski | 2013-06-20 | 12 | -25/+0 |
| | | | | | | | | They were moved to actionview/ and are not used in actionpack | ||||
* | | Remove unneeded test fixtures in AV | Piotr Sarnacki | 2013-06-20 | 108 | -281/+0 |
| | | |||||
* | | Remove unneeded files | Piotr Sarnacki | 2013-06-20 | 2 | -50/+0 |
| | | |||||
* | | Move template tests from actionpack to actionview | Piotr Sarnacki | 2013-06-20 | 52 | -16208/+0 |
| | | |||||
* | | Added test for link_to_unless to make sure the result consistency. | dtaniwaki | 2013-06-16 | 1 | -0/+5 |
| | | |||||
* | | Use a case insensitive URI Regexp for #asset_path | David Celis | 2013-06-16 | 1 | -2/+5 |
|/ | | | | | | | | | | | | | Context: https://gist.github.com/radar/5793814 The `URI_REGEXP` that various AssetUrl helpers use is currently case sensitive when checking for a URI scheme. This means if you try to pass a URL like `HTTP://www.example.com/path/to/image.jpg`, you end up with a bogus asset path: `/assets/HTTP://www.example.com/path/to/image.jpg`. URLs are case insensitive, so this regexp should be as well. Signed-off-by: David Celis <me@davidcel.is> | ||||
* | collection tags accept html attributes as the last element of collection | Vasiliy Ermolovich | 2013-06-16 | 1 | -0/+16 |
| | |||||
* | Update the HTML boolean attributes per the HTML 5.1 spec | Alex Peattie | 2013-06-14 | 1 | -2/+2 |
| | | | | | | | - Add attributes `allowfullscreen`, `default`, `inert`, `sortable`, `truespeed`, `typemustmatch`. - Fix attribute `seamless` (previously misspelled `seemless`). - Use `assert_dom_equal` instead of `assert_equal` in test. | ||||
* | Don't set X-UA-Compatible header by default | Guillermo Iguaran | 2013-06-13 | 1 | -3/+1 |
| | | | | | | We are setting this header to chrome=1 for Chrome Frame and this will be retired soon. Check http://blog.chromium.org/2013/06/retiring-chrome-frame.html for details | ||||
* | Warning removed for Minitest | Arun Agrawal | 2013-06-12 | 1 | -1/+1 |
| | |||||
* | Fix mismatching variable names when using an underscore | Drew Ulmer | 2013-06-04 | 1 | -13/+41 |
| | | | | | | | | The ERBTracker template digest helper class was using a regex to match render calls and it was incorrectly not matching against variables with underscores in the name. This caused it to use the wrong regex match data to populate the template dependency. Because underscore is a valid character for a variable, this fixes the ERBTracker to match it properly. | ||||
* | Raise ArgumentError to fix Rails 4.1 deprecation warn message. | kennyj | 2013-06-02 | 1 | -5/+7 |
| | |||||
* | Remove Mime::Type#verify_request? and Mime::Type.browser_generated_types ↵ | kennyj | 2013-06-02 | 2 | -21/+0 |
| | | | | were deprecated. | ||||
* | Remove ActionView::Template#mime_type was deprecated. | kennyj | 2013-06-01 | 1 | -7/+0 |
| | |||||
* | Remove ActionController::RecordIdentifier was deprecated. | kennyj | 2013-06-01 | 2 | -63/+3 |
| | |||||
* | Remove :confirm and :disable_with options from ↵ | kennyj | 2013-06-01 | 1 | -28/+0 |
| | | | | ActionView::Helpers::FormTagHelper were deprecated. | ||||
* | Remove link_to_function / button_to_function from ↵ | kennyj | 2013-06-01 | 1 | -42/+0 |
| | | | | ActionView::Helpers::JavascriptHelper were deprecated. | ||||
* | Remove :confirm and :disable_with options for ↵ | kennyj | 2013-06-01 | 1 | -75/+0 |
| | | | | ActionView::Helpers::UrlHelper#link_to and #button_to were deprecated. | ||||
* | `RoutesInspector` deals with routes using regexp as `:controller` option | Yves Senn | 2013-05-30 | 1 | -0/+9 |
| | |||||
* | Running isolated tests | Arun Agrawal | 2013-05-28 | 1 | -15/+0 |
| | | | | These stopped running don't know when. But these should be running now. | ||||
* | Add has_named_route? to the mapper API | José Valim | 2013-05-20 | 1 | -0/+13 |
| | |||||
* | Fix named routing regression from 3.2.13 | schneems | 2013-05-16 | 1 | -0/+11 |
| | | | | | | | | | | | | | | | | | | | | | | When named route that is nested is used in 3.2.13 Example `routes.rb`: ``` resources :nested do resources :builder, :controller => 'nested/builder' end ``` In 3.2.12 and 3.2.12 this named route would work: ``` nested_builder_path(:last_step, :nested_id => "foo") ``` Generating a url that looks like `/nested/foo/builder/last_step`. This PR fixes the regression when building urls via the optimized helper. Any explicit keys set in the options are removed from the list of implicitly mapped keys. Not sure if this is exactly how the original version worked, but this fixes this use case regression. | ||||
* | Revert "Integration tests support the OPTIONS http method" | Rafael Mendonça França | 2013-05-16 | 1 | -17/+1 |
| | | | | | | | | | | | | | | This reverts commit ad46884af567d6f8d6d8d777f372c39e81a560ba. Conflicts: actionpack/CHANGELOG.md actionpack/lib/action_dispatch/testing/integration.rb actionpack/test/controller/integration_test.rb Reason: It will conflict with a lot of test cases. Better to call `process` directly since this is a very uncommon HTTP method. Fixes #10638. | ||||
* | Merge pull request #10636 from seanwalbran/fix-asset-host-proc-returning-nil | Rafael Mendonça França | 2013-05-16 | 1 | -0/+11 |
|\ | | | | | fix rails4 regression with asset_host Proc returning nil | ||||
| * | restore ability to return nil from asset_host proc, add test | Sean Walbran | 2013-05-15 | 1 | -0/+11 |
| | | |||||
* | | Revert "Add the options method to action_controller testcase." | Rafael Mendonça França | 2013-05-16 | 1 | -5/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0303c2325fab253adf5e4a0b738cb469c048f008. Conflicts: actionpack/lib/action_controller/test_case.rb REASON: It will conflict with a lot of test cases. Better to call `process` directly since this is a very uncommon HTTP method. Fixes #10638. | ||||
* | | Merge pull request #10591 from acapilleri/cond_callback_test | Aaron Patterson | 2013-05-15 | 1 | -0/+13 |
|\ \ | |/ |/| | add test for skip_before_filter with condition | ||||
| * | add test for skip_before_filter with condition | Angelo capilleri | 2013-05-13 | 1 | -0/+13 |
| | | |||||
* | | use public api for testing rather than grabbing instance variables | Aaron Patterson | 2013-05-14 | 1 | -1/+1 |
|/ | |||||
* | Replace multi_json with json | Erik Michaels-Ober | 2013-05-11 | 1 | -1/+1 |
| | |||||
* | Fix that JSON and XML exception responses should give the HTTP error message ↵ | Jeremy Kemper | 2013-05-08 | 1 | -2/+2 |
| | | | | for their status, by default, not the message from the underlying exception | ||||
* | Test that #fresh_when accepts an array | Brandon Keepers | 2013-05-07 | 1 | -2/+19 |
| | | | | https://github.com/rails/etagger/pull/3 | ||||
* | Allow numbers in partial name for digesting | Bryan Ricker | 2013-05-06 | 2 | -1/+8 |
| | | | | | | | | | | Add failing test for template with number at the end Use \w for RENDER_DEPENDENCY regex Spacing Add CHANGELOG entry | ||||
* | Fix failing AP test | Carlos Antonio da Silva | 2013-05-01 | 1 | -1/+1 |
| | |||||
* | Merge pull request #9857 from yyyc514/bad_params_should_400 | Aaron Patterson | 2013-04-30 | 2 | -0/+7 |
|\ | | | | | failure to parse params should trigger a 400 Bad Request | ||||
| * | failure to parse params should trigger a 400 Bad Request | Josh Goebel | 2013-03-21 | 2 | -0/+7 |
| | |