Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | remove duplicate usage of Rack::Utils.status_code in ↵ | Rafael Magana | 2012-05-10 | 1 | -1/+1 | |
|/ / | | | | | | | LogSubscriber#process_action | |||||
* | | If content_type is explicitly passed to the :head method use the value or ↵ | Kunal Shah | 2012-05-08 | 1 | -1/+2 | |
| | | | | | | | | fallback | |||||
* | | remove .new from raise ActionController::UnknownFormat | Steven Soroka | 2012-05-06 | 1 | -1/+1 | |
| | | ||||||
* | | Raise a rescuable exception when Rails doesn't know what to do with the ↵ | Steven Soroka | 2012-05-06 | 2 | -2/+4 | |
| | | | | | | | | format, rather than responding with a head :not_acceptable (406) | |||||
* | | Use Hash#fetch instead of has_key? check | Carlos Antonio da Silva | 2012-05-04 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #4445 from nragaz/role_based_params_wrapping | José Valim | 2012-05-04 | 1 | -2/+3 | |
| | | | | | | | | specify a role for identifying accessible attributes when wrapping params | |||||
* | | Improve assert_template layout checking | Alexey Vakhov | 2012-05-04 | 1 | -3/+15 | |
| | | ||||||
* | | Fix assert_template :layout => nil assertion | Alexey Vakhov | 2012-05-04 | 1 | -2/+5 | |
| | | ||||||
* | | Fix assert_template assertion with :layout option | Alexey Vakhov | 2012-05-04 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #6148 from twinturbo/head-fix | José Valim | 2012-05-03 | 1 | -1/+21 | |
|\ \ | | | | | | | Make ActionController#head pass rack-link | |||||
| * | | Remove content-length as well | twinturbo | 2012-05-03 | 1 | -2/+3 | |
| | | | ||||||
| * | | Make ActionController#head pass rack-link | twinturbo | 2012-05-03 | 1 | -1/+20 | |
| | | | ||||||
* | | | added proc evaluation for action cache's layout parameter | Nico | 2012-05-02 | 1 | -2/+4 | |
| | | | ||||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-05-01 | 2 | -2/+2 | |
|\ \ \ | |/ / |/| | | ||||||
| * | | Code-format references to config settings | Mark Rushakoff | 2012-04-27 | 1 | -1/+1 | |
| | | | ||||||
| * | | More using <tt>x</tt> instead of +x+ when the latter renders improperly. | Mark Rushakoff | 2012-04-27 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #6078 from lest/patch-4 | José Valim | 2012-04-30 | 1 | -18/+12 | |
|\ \ \ | | | | | | | | | allow send_file/send_data to skip disposition header, closes #2973 | |||||
| * | | | no need to dup options in send_data | Sergey Nartimov | 2012-04-30 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | options aren't modified inside send_file_headers! | |||||
| * | | | allow send_file/send_data to skip disposition header, closes #2973 | Sergey Nartimov | 2012-04-30 | 1 | -17/+11 | |
| | | | | ||||||
* | | | | Merge pull request #6034 from ↵ | Piotr Sarnacki | 2012-04-30 | 1 | -1/+0 | |
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | willbryant/flash_must_not_load_session_on_every_request_master Fix the Flash middleware loading the session on every request | |||||
| * | | | fix the Flash middleware loading the session on every request (very ↵ | Will Bryant | 2012-04-28 | 1 | -1/+0 | |
| |/ / | | | | | | | | | | dangerous especially with Rack::Cache), it should only be loaded when the flash method is called | |||||
* | | | Add note about using 303 See Other for XHR requests other than GET/POST | Andrew White | 2012-04-30 | 1 | -0/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | IE since version 6 and recently Chrome and Firefox have started following 302 redirects from XHR requests other than GET/POST using the original request method. This can lead to DELETE requests being redirected amongst other things. Although it doesn't directly affect the Rails framework since it doesn't return a 302 redirect to any non-GET/POST request a note has been added to raise awareness of the issue. Some references: Original article from @technoweenie: http://techno-weenie.net/2011/8/19/ie9-deletes-stuff/ Hacker News discussion of the article: http://news.ycombinator.com/item?id=2903493 WebKit bug report: https://bugs.webkit.org/show_bug.cgi?id=46183 Firefox bug report and changeset: https://bugzilla.mozilla.org/show_bug.cgi?id=598304 https://hg.mozilla.org/mozilla-central/rev/9525d7e2d20d Chrome bug report: http://code.google.com/p/chromium/issues/detail?id=56373 HTTPbis bug report and changeset: http://trac.tools.ietf.org/wg/httpbis/trac/ticket/160 http://trac.tools.ietf.org/wg/httpbis/trac/changeset/1428 Roy T. Fielding's history of the issue: http://ftp.ics.uci.edu/pub/ietf/http/hypermail/1997q3/0611.html Automated browser tests for the issue: http://www.mnot.net/javascript/xmlhttprequest/ Fixes #4144 | |||||
* | | | Fix controller_class_name for anonymous controllers. | Michael Schuerig | 2012-04-29 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #6017 from larzconwell/remove_sanitize_dom_id | Jeremy Kemper | 2012-04-29 | 1 | -6/+1 | |
|\ \ \ | | | | | | | | | Removed the sanitize_dom_id method | |||||
| * | | | Removed the sanitize_dom_id method because HTML5 doctype let's us use ↵ | Larz Conwell | 2012-04-27 | 1 | -6/+1 | |
| | | | | | | | | | | | | | | | | anything except nothing and whitespace for id's | |||||
* | | | | Don't convert params if the request isn't HTML - fixes #5341 | Andrew White | 2012-04-29 | 1 | -6/+18 | |
| | | | | ||||||
* | | | | Avoid calling content type multiple times | José Valim | 2012-04-29 | 1 | -4/+6 | |
| | | | | ||||||
* | | | | Merge pull request #2321 from omjokine/master | José Valim | 2012-04-29 | 1 | -1/+4 | |
|\ \ \ \ | | | | | | | | | | | JSONP should use mimetype application/javascript | |||||
| * \ \ \ | Merge remote-tracking branch 'upstream/master' | Olli Jokinen | 2011-12-01 | 29 | -102/+134 | |
| |\ \ \ \ | ||||||
| * | | | | | Change mimetype to Mime::JS if JSONP is used | Olli Jokinen | 2011-07-28 | 1 | -1/+4 | |
| | | | | | | ||||||
* | | | | | | Merge session arg with existing session instead of overwriting | Andrew White | 2012-04-28 | 1 | -1/+1 | |
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This may break existing tests that are asserting the whole session contents but should not break existing tests that are asserting individual keys - e.g: class SomeControllerTest < ActionController::TestCase setup do session['user_id'] = 1 end test "some test" do get :some_action, nil, { 'another_var' => 2 } # This assertion will now fail assert_equal({ 'another_var' => 2 }, session) # This assertion will still pass assert_equal 2, session['another_var] end end Fixes #1529. | |||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-28 | 2 | -4/+4 | |
|\ \ \ \ \ | |_|_|/ / |/| | | | | ||||||
| * | | | | Use <tt>Foo::Bar</tt> instead of +Foo::Bar+ | Mark Rushakof | 2012-04-27 | 2 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The latter doesn't render as code in HTML output. Regex used in Rubymine to locate the latter form: (\+)(:*\w+:(?::|\w)+)(\+) | |||||
* | | | | | remove unnecessary else statement | Vasiliy Ermolovich | 2012-04-26 | 1 | -3/+3 | |
|/ / / / | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-18 | 1 | -1/+1 | |
|\ \ \ \ | ||||||
| * | | | | Fix HTML sanitizer allowed_css_properties comment | Alexey Vakhov | 2012-04-13 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Remove obsolete code | Alexey Vakhov | 2012-04-12 | 1 | -3/+0 | |
| | | | | | ||||||
* | | | | | #send_file leans on Rack::Sendfile to X-Accel-Redirect the file's path, so ↵ | Jeremy Kemper | 2012-04-12 | 1 | -1/+21 | |
|/ / / / | | | | | | | | | | | | | opening the file to set the response body is wasteful. Set a FileBody wrapper instead that responds to to_path and streams the file if needed. | |||||
* | | | | Merge pull request #5748 from ai/no_type_in_html5 | Jeremy Kemper | 2012-04-08 | 1 | -1/+1 | |
|\ \ \ \ | | | | | | | | | | | Remove unnecessary in HTML 5 type attribute with default value | |||||
| * | | | | Remove unnecessary in HTML 5 type attribute with default value | Andrey A.I. Sitnik | 2012-04-05 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-05 | 1 | -0/+1 | |
|\ \ \ \ \ | ||||||
| * | | | | | Add AC record identifier example with not-persisted object | Alexey Vakhov | 2012-04-05 | 1 | -0/+1 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #4446 from ayamomiji/patch-1 | José Valim | 2012-04-05 | 1 | -1/+1 | |
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | prevent the cache sweeper ignores NoMethodError | |||||
| * | | | | | prevent the cache sweeper ignores NoMethodError | ayaya | 2012-03-29 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #5716 from jurriaan/tr-gsub-cleanup | Santiago Pastorino | 2012-04-04 | 3 | -3/+3 | |
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | Updated/changed unneeded tr/gsubs | |||||
| * | | | | | Updated/changed useless tr/gsubs | Jurriaan Pruis | 2012-04-03 | 3 | -3/+3 | |
| | | | | | | ||||||
* | | | | | | Fix AC responder example | Alexey Vakhov | 2012-04-04 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Remove non-obligatory params in AC respond_to examples | Alexey Vakhov | 2012-04-03 | 1 | -2/+2 | |
|/ / / / / | ||||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-01 | 1 | -1/+0 | |
|\ \ \ \ \ | ||||||
| * | | | | | CSRF messages are no longer controlled by 422.html because ↵ | Tony Primerano | 2012-03-28 | 1 | -1/+0 | |
| |/ / / / | | | | | | | | | | | | | | | | InvalidAuthenticityToken is not raised |