Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | Merge pull request #6083 from brainopia/remove_unused_closed_ivars | José Valim | 2012-04-30 | 2 | -2/+0 | |
|\ \ \ \ \ | |/ / / / |/| | | | | Remove a couple of unused ivars left from previous refactoring | |||||
| * | | | | Remove unused ivars left from close checks | brainopia | 2012-04-30 | 2 | -2/+0 | |
| |/ / / | | | | | | | | | | | | | These ivars were missed in d142572567 when close checks were removed | |||||
* | | | | Merge pull request #6034 from ↵ | Piotr Sarnacki | 2012-04-30 | 1 | -6/+3 | |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | 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 | -6/+3 | |
| |/ / / | | | | | | | | | | | | | dangerous especially with Rack::Cache), it should only be loaded when the flash method is called | |||||
* | | | | Merge pull request #6074 from mark-rushakoff/unused-variables | Jeremy Kemper | 2012-04-29 | 1 | -1/+0 | |
|\ \ \ \ | | | | | | | | | | | Remove some unused variable assignments | |||||
| * | | | | Remove unused assignments | Mark Rushakoff | 2012-04-29 | 1 | -1/+0 | |
| |/ / / | ||||||
* | | | | Restore interpolation of path option in redirect routes | Andrew White | 2012-04-29 | 1 | -0/+10 | |
| | | | | ||||||
* | | | | Escape interpolated params when redirecting - fixes #5688 | Andrew White | 2012-04-29 | 1 | -1/+7 | |
|/ / / | ||||||
* | | | Shadowing variable warning removed | Arun Agrawal | 2012-04-26 | 1 | -3/+3 | |
| | | | ||||||
* | | | Allow loading external route files from the router | Jose and Yehuda | 2012-04-25 | 2 | -0/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature enables the ability to load an external routes file from the router via: draw :filename External routes files go in +config/routes+. This feature works in both engines and applications. | |||||
* | | | Merge pull request #5980 from gazay/valid_ips | Jeremy Kemper | 2012-04-25 | 1 | -22/+68 | |
|\ \ \ | | | | | | | | | Remote ip logic and validation. IPv6 support. | |||||
| * | | | Valid ips v4 and v6. Right logic for working with X-FORWARDED-FOR header and ↵ | Alexey Gaziev | 2012-04-25 | 1 | -22/+68 | |
| |/ / | | | | | | | | | | tests. | |||||
* | | | Remove unnecessary empty line | José Valim | 2012-04-25 | 1 | -1/+0 | |
| | | | ||||||
* | | | Remove default match without specified method | Jose and Yehuda | 2012-04-24 | 1 | -2/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964 | |||||
* | | | Merge pull request #5961 from bogdan/routes | Jeremy Kemper | 2012-04-24 | 1 | -20/+26 | |
|\ \ \ | | | | | | | | | Refactor ActionDispatch::HTTP::Url#url_for | |||||
| * | | | ActionDispatch::HTTP::Url#url_for refactor method | Bogdan Gusiev | 2012-04-24 | 1 | -20/+26 | |
| | | | | | | | | | | | | | | | | | | | | Separated right side url generation(before query string) from left side url generation(after query string) | |||||
* | | | | Refactor hash creation in routeset | Carlos Antonio da Silva | 2012-04-24 | 1 | -3/+4 | |
|/ / / | ||||||
* | | | RouteSet: decomplecting a way to handle positional args | Bogdan Gusiev | 2012-04-24 | 1 | -26/+20 | |
| | | | ||||||
* | | | RouteSet: simplify routes helpers generation code | Bogdan Gusiev | 2012-04-24 | 1 | -13/+21 | |
| | | | ||||||
* | | | Add missing requires in routes | Bogdan Gusiev | 2012-04-23 | 2 | -0/+3 | |
|/ / | ||||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-21 | 1 | -2/+2 | |
|\ \ | ||||||
| * | | Fix formatting | Oscar Del Ben | 2012-04-18 | 1 | -2/+2 | |
| | | | ||||||
* | | | Freeze the middleware stack after it's built | Jeremy Kemper | 2012-04-20 | 1 | -1/+1 | |
|/ / | | | | | | | | | | | So apps that accidentally add middlewares later aren't unwittingly dumping them in a black hole. Closes #5911 | |||||
* | | let's keep the slash in the return value instead of the path variable [ci skip] | Vijay Dev | 2012-04-18 | 1 | -2/+2 | |
| | | ||||||
* | | Warn about do end syntax in redirect option | Oscar Del Ben | 2012-04-17 | 1 | -0/+3 | |
| | | ||||||
* | | Fix redirect with block example | Alexey Vakhov | 2012-04-17 | 1 | -3/+3 | |
| | | ||||||
* | | Fix example in flash middleware | Alexey Vakhov | 2012-04-17 | 1 | -1/+1 | |
| | | ||||||
* | | Fix comment in AD http response | Alexey Vakhov | 2012-04-13 | 1 | -1/+1 | |
| | | | | | | | | response in integration test accessible without @ via attr_reader | |||||
* | | Revert "Merge pull request #5750 from ahoward/master" | Santiago Pastorino | 2012-04-08 | 1 | -10/+1 | |
| | | | | | | | | | | | | | | This reverts commit 174cf8b22064c3999dfa685014d5b31ac993bb54, reversing changes made to 7ecd6a731bd60665bc6de94095137f0b2c4ada2a. The reverted commit improved the performance in the wrong place, now we have added this 6ddbd1844a6fd6aca2992f5f75c9f605cf89808f improvement. | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-07 | 1 | -1/+1 | |
|\ \ | ||||||
| * | | Fix example action dispatch in mime type | Alexey Vakhov | 2012-04-07 | 1 | -1/+1 | |
| | | | ||||||
* | | | documents the contract for the argument of AD::Routing::RouteSet#url_for | Xavier Noria | 2012-04-05 | 1 | -0/+1 | |
| | | | ||||||
* | | | Make url_for work when option is nil and default_url_options has | Rafael Mendonça França | 2012-04-05 | 1 | -1/+1 | |
| | | | | | | | | | | | | stringified keys | |||||
* | | | Fix url_for when options is nil | Rafael Mendonça França | 2012-04-05 | 1 | -1/+1 | |
|/ / | | | | | | | | | RouteSet#url_for was modifying the options hash that belong to the caller | |||||
* | | url_for: avoid a nil check. unroll a loop. handle most common case 1st. | ara.t.howard | 2012-04-05 | 1 | -18/+13 | |
| | | ||||||
* | | carefully and quickly merge url_for options | ara.t.howard | 2012-04-04 | 1 | -1/+17 | |
| | | ||||||
* | | Merge pull request #5716 from jurriaan/tr-gsub-cleanup | Santiago Pastorino | 2012-04-04 | 2 | -3/+3 | |
|\ \ | | | | | | | Updated/changed unneeded tr/gsubs | |||||
| * | | Updated/changed useless tr/gsubs | Jurriaan Pruis | 2012-04-03 | 2 | -3/+3 | |
| | | | ||||||
* | | | No need to extend ActiveSupport::Extend here | Oscar Del Ben | 2012-04-04 | 1 | -2/+0 | |
|/ / | ||||||
* | | Merge pull request #5694 from hardipe/master | Santiago Pastorino | 2012-04-01 | 1 | -4/+0 | |
|\ \ | | | | | | | Tidy up cookies.rb a bit | |||||
| * | | Useless/Confusing method definition removed | hardi | 2012-04-01 | 1 | -4/+0 | |
| | | | ||||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-01 | 1 | -1/+1 | |
|\ \ \ | |/ / |/| | | ||||||
| * | | :success includes the whole 200 range, not just 200. | Erich Menge | 2012-04-01 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | # File lib/rack/response.rb, line 114 114: def successful?; @status >= 200 && @status < 300; end | |||||
* | | | Remove not needed requires | Santiago Pastorino | 2012-03-31 | 1 | -1/+0 | |
| | | | ||||||
* | | | Remove the leading \n added by textarea on assert_select | Santiago Pastorino | 2012-03-30 | 1 | -0/+1 | |
| | | | ||||||
* | | | adds delegetion for eof? to AD::Http::UploadedFile | Jens Fahnenbruck | 2012-03-27 | 1 | -1/+1 | |
|/ / | | | | | | | | | if you want to read the file you may need to ask if there is something to read from | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-03-27 | 1 | -0/+19 | |
|\ \ | ||||||
| * | | add missing do [ci skip] | Vijay Dev | 2012-03-27 | 1 | -1/+1 | |
| | | | ||||||
| * | | added shallow_prefix option description in documantation | Lukasz Sarnacki | 2012-03-27 | 1 | -0/+19 | |
| | | | | | | | | | | | | It is in response for confusion in issue #5301 | |||||
* | | | Return the same session data object when setting session id | Carlos Antonio da Silva | 2012-03-26 | 1 | -1/+2 | |
| | | |