Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #3617 from indirect/remote_ip | José Valim | 2011-11-13 | 2 | -53/+55 |
|\ | | | | | refactor RemoteIp middleware | ||||
| * | defer calculating the remote IP until requested | Andre Arko | 2011-11-12 | 1 | -24/+36 |
| | | |||||
| * | refactor RemoteIp middleware | Andre Arko | 2011-11-11 | 2 | -55/+45 |
| | | | | | | | | | | | | | | | | | | | | | | - return the last forwarded IP before REMOTE_ADDR to handle proxies - remove completely superfluous RemoteIpGetter class - remove duplication of trusted proxies regexp - remove unused constant from Request - move comments from Request to where they are actually relevant - edit comments for clarity of purpose The original code (confusingly) tried to return REMOTE_ADDR both at the beginning and the end of the chain of options. Since REMOTE_ADDR is _always_ set, this is kind of silly. This change leaves REMOTE_ADDR as the last option, so that proxied requests will be assigned the correct remote IP address. | ||||
* | | Unneeded require memoizable | Akira Matsuda | 2011-11-12 | 1 | -2/+0 |
|/ | |||||
* | Fix trouble using :subdomain in development environment when using ↵ | Bradford Folkens | 2011-11-08 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | numeric addresses. See-also pull request #3561 from 3-1-stable Otherwise the following occurs: TypeError: can't convert nil into String /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/http/url.rb:75:in host_or_subdomain_and_domain' /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/http/url.rb:37:in url_for' /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/routing/url_for.rb:147:in test_subdomain_may_be_accepted_with_numeric_host' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/testing/setup_and_teardown.rb:67:in run' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/callbacks.rb:426:in send' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/callbacks.rb:81:in run' | ||||
* | Fix small typos in routing docs | Alexey Vakhov | 2011-11-03 | 1 | -1/+3 |
| | |||||
* | Fix typo in Dispatcher#controller documentation | Aviv Ben-Yosef | 2011-11-01 | 1 | -1/+1 |
| | |||||
* | Fix typo in constraints method documentation | Alexey Vakhov | 2011-11-01 | 1 | -1/+1 |
| | |||||
* | Adds missing closing regex slashes. | mjy | 2011-10-25 | 1 | -2/+2 |
| | |||||
* | Add ActionDispatch::Session::CacheStore as a generic way of storing sessions ↵ | Brian Durand | 2011-10-21 | 1 | -0/+50 |
| | | | | in a cache. | ||||
* | Remove the unneeded `\d` when sanitizing `X-Request-Id`. | Marc Bowes | 2011-10-20 | 1 | -1/+1 |
| | |||||
* | Load object/blank and make use of presence. | José Valim | 2011-10-19 | 1 | -2/+3 |
| | |||||
* | Make tests run on 1.8.x, add integration setup. | José Valim | 2011-10-19 | 1 | -3/+2 |
| | |||||
* | Blah, SecureRandom#uuid is not supported in 1.8.7 -- cant wait for Rails 4.0 ↵ | David Heinemeier Hansson | 2011-10-19 | 1 | -1/+1 |
| | | | | to drop compatibility with 1.8.x | ||||
* | Require missing string access dependency. | José Valim | 2011-10-19 | 1 | -0/+1 |
| | |||||
* | Require securerandom as it is the proper dependency. | José Valim | 2011-10-19 | 1 | -1/+1 |
| | |||||
* | Added X-Request-Id tracking and TaggedLogging to easily log that and other ↵ | David Heinemeier Hansson | 2011-10-19 | 2 | -0/+48 |
| | | | | production concerns | ||||
* | Remove superfluous assignment in cookies | Alexey Vakhov | 2011-10-18 | 1 | -1/+1 |
| | |||||
* | / is allowed in URI fragments | Jeremy Kemper | 2011-10-13 | 1 | -1/+1 |
| | |||||
* | Leave escaping up to Journey | Jeremy Kemper | 2011-10-13 | 1 | -4/+3 |
| | |||||
* | Add some implementation docs. closes #3298. closes #2509. | José Valim | 2011-10-12 | 1 | -0/+4 |
| | |||||
* | allow shorthand routes with nested optional parameters | Diego Carrion | 2011-10-10 | 1 | -3/+3 |
| | |||||
* | fix require | Aaron Patterson | 2011-10-06 | 1 | -1/+1 |
| | |||||
* | Clean up subdomain code a bit. | José Valim | 2011-10-04 | 1 | -2/+2 |
| | |||||
* | :subdomain can now be specified with a value of false in url_for, allowing ↵ | Kamil Sobieraj | 2011-10-04 | 2 | -5/+8 |
| | | | | for subdomain(s) removal from the host during link generation. Closes #2025 | ||||
* | TestCase should respect the view_assigns API instead of pulling variables on ↵ | José Valim | 2011-10-02 | 1 | -6/+1 |
| | | | | its own. | ||||
* | Implement Mime::Type#respond_to? (consistently with #method_missing) | Evgeniy Dolzhenko | 2011-09-25 | 1 | -0/+4 |
| | |||||
* | Revert "Make process reuse the env var passed as argument" | Santiago Pastorino | 2011-09-24 | 1 | -4/+4 |
| | | | | This reverts commit 0e4748cd415660eb91e63d50aa15cdd027c612dd. | ||||
* | Deprecate passing the template handler in the template name. | José Valim | 2011-09-22 | 1 | -2/+2 |
| | | | | | | For example, calling hello.erb is now deprecated. Since Rails 3.0 passing the handler had no effect whatsover. This commit simply deprecates such cases so we can clean up the code in later releases. | ||||
* | Fix named routes modifying arguments | Pawel Pierzchala | 2011-09-22 | 1 | -3/+4 |
| | |||||
* | removing backwards compatibility module | Aaron Patterson | 2011-09-12 | 3 | -9/+8 |
| | |||||
* | partially expand the parameters to `match` | Aaron Patterson | 2011-09-12 | 1 | -3/+3 |
| | |||||
* | unfactor the Route class to private factory methods | Aaron Patterson | 2011-09-12 | 2 | -45/+42 |
| | |||||
* | reduce dependencies of external objects in the Route class | Aaron Patterson | 2011-09-12 | 2 | -16/+18 |
| | |||||
* | delete unused code, pass path explicitly to journey | Aaron Patterson | 2011-09-12 | 2 | -28/+7 |
| | |||||
* | all routes can be stored in the Journey Routes object | Aaron Patterson | 2011-09-12 | 1 | -5/+4 |
| | |||||
* | clear! does not need to be called from initialize | Aaron Patterson | 2011-09-12 | 1 | -4/+3 |
| | |||||
* | reuse the route collection and formatter by clearing them | Aaron Patterson | 2011-09-12 | 1 | -6/+10 |
| | |||||
* | Instantiate each part of our routing system: | Aaron Patterson | 2011-09-12 | 1 | -7/+8 |
| | | | | | | * A collection of Routes * The Router, which consults the collection of routes * A formatter that consults the collection of routes | ||||
* | stop freezing the routes | Aaron Patterson | 2011-09-12 | 1 | -1/+0 |
| | |||||
* | stop using a hash for parameterizing | Aaron Patterson | 2011-09-12 | 1 | -11/+9 |
| | |||||
* | Switching rack-mount to journey. | Aaron Patterson | 2011-09-12 | 1 | -1/+2 |
| | |||||
* | Merge pull request #2499 from akaspick/assert_select_email_fix | Jon Leighton | 2011-09-07 | 1 | -2/+2 |
|\ | | | | | Fix assert_select_email to work on non-multipart emails as well as converting the Mail::Body to a string to prevent errors. | ||||
| * | fix assert_select_email to work on non-multipart emails as well as ↵ | Andrew Kaspick | 2011-08-11 | 1 | -2/+2 |
| | | | | | | | | converting the Mail::Body to a string to prevent errors. | ||||
* | | Merge pull request #2577 from rails-noob/master | Santiago Pastorino | 2011-09-06 | 1 | -1/+3 |
|\ \ | | | | | | | Fix double slash at start of paths when mounting an engine at the root. | ||||
| * | | Fix bug #2579. | rails-noob | 2011-09-06 | 1 | -1/+3 |
| | | | | | | | | | | | | Avoids double slash at start of paths when mounting an engine at the root. | ||||
* | | | if ... nil? is more expensive than unless | Milan Dobrota | 2011-09-04 | 1 | -1/+1 |
| | | | |||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-09-02 | 1 | -6/+6 |
|\ \ \ | |||||
| * | | | Fix name of parent resource params when declaring nested resources at routes | Carlos Paramio | 2011-09-01 | 1 | -6/+6 |
| | | | | |||||
* | | | | CookieJar is enumerable. fixes #2795 | Aaron Patterson | 2011-09-01 | 1 | -0/+5 |
| | | | |