aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #3500 from olivierlacan/error-whitespaceJon Leighton2011-11-161-0/+1
|\ | | | | CSS fix to prevent error output from being breaking out of body element.
| * CSS fix to prevent error output from being breaking out of body element.Olivier Lacan2011-11-031-0/+1
| | | | | | | | | | | | | | | | | | Using the white-space: pre-wrap adds extra line breaks to prevent the text from breaking out of the element's box. In this case single line output can be extremely long, breaking out the <body> element. See for reference: http://www.quirksmode.org/css/whitespace.html Before: http://link.olivierlacan.com/BVU4 After: http://link.olivierlacan.com/BUfM
* | Switch from marshal format to plain text for the encoding conversions dump. ↵Jon Leighton2011-11-163-6/+95
| | | | | | | | This is for windows compatibility. Fixes #3644.
* | Revert "Merge pull request #3640 from indirect/remote_ip"Jon Leighton2011-11-152-8/+6
| | | | | | | | | | | | | | | | This reverts commit 6491aadc525b8703708e0fd0fbf05bd436a47801, reversing changes made to 83bf0b626cf2134260903e57d74f67de57384073. See https://github.com/rails/rails/pull/3640#issuecomment-2752761 for explanation.
* | GetIp#to_s should never return nil. That's icky.Andre Arko2011-11-142-6/+8
| |
* | Return the calculated remote_ip or ipAndre Arko2011-11-141-1/+2
| | | | | | This was an especially nasty bug introduced in 317f4e2, by the way that an instance of GetIp is not nil, but GetIp#to_s could sometimes return nil. Gross, huh?
* | memoize the relatively expensive remote IP codeAndre Arko2011-11-141-1/+7
| |
* | cleaner namesAndre Arko2011-11-141-6/+6
| |
* | remove ignored flag, fixes warningsAndre Arko2011-11-131-1/+1
| |
* | turns out the tests expect remote_addrs.firstAndre Arko2011-11-131-1/+1
| |
* | correctly raise IpSpoofAttackError messageAndre Arko2011-11-131-2/+2
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-131-2/+2
|\ \
| * | Cosmetic fix in number_to_currency docsAlexey Vakhov2011-11-101-2/+2
| | |
* | | Merge pull request #3617 from indirect/remote_ipJosé Valim2011-11-132-53/+55
|\ \ \ | | | | | | | | refactor RemoteIp middleware
| * | | defer calculating the remote IP until requestedAndre Arko2011-11-121-24/+36
| | | |
| * | | refactor RemoteIp middlewareAndre Arko2011-11-112-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 memoizableAkira Matsuda2011-11-121-2/+0
|/ / /
* | | Merge pull request #3613 from mitio/fix-actionpack-responder-docsXavier Noria2011-11-111-4/+4
|\ \ \ | | | | | | | | Fix wording and minor typos in the Responder RDoc
| * | | Fix wording and minor typos in the Responder RDocDimitar Dimitrov2011-11-111-4/+4
| | | |
* | | | Merge pull request #3592 from avakhov/av-highlight-regexpJosé Valim2011-11-101-1/+1
|\ \ \ \ | | | | | | | | | | Fix and simplify highlight regexp
| * | | | Fix and simplify highlight regexpAlexey Vakhov2011-11-101-1/+1
| | |/ / | |/| |
* | | | Merge pull request #3581 from amatsuda/metal_response_body_19José Valim2011-11-101-1/+7
|\ \ \ \ | |/ / / |/| | | AC::Metal#response_body= stores different value in Ruby 1.8 and 1.9
| * | | Fix AC::Metal#response_body= to store same value on Ruby 1.8 and 1.9Akira Matsuda2011-11-071-1/+7
| | | | | | | | | | | | | | | | This was because String#respond_to?(:each) differs in 1.8 and 1.9
* | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-082-13/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: railties/guides/source/active_record_querying.textile
| * | | | Fix small typo in link_to_function docAlexey Vakhov2011-11-081-1/+1
| | | | |
| * | | | Fixed stale doc in AbstractController::LayoutsDalibor Nasevic2011-11-061-12/+5
| |/ / /
* | | | Fix trouble using :subdomain in development environment when using ↵Bradford Folkens2011-11-081-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'
* | | | DrbStore removed from cache backendsIvan Evtukhovich2011-11-071-1/+0
| | | |
* | | | fix incorrect code exampleVijay Dev2011-11-071-1/+1
|/ / /
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-061-2/+2
|\ \ \
| * | | Docs cosmetic fixes in Action View form_helper.rbAlexey Vakhov2011-11-061-2/+2
| | | |
* | | | RUBY_ENGINE is not defined on 1.8Jon Leighton2011-11-061-1/+1
| | | |
* | | | Implement a workaround for a bug in ruby-1.9.3p0.Jon Leighton2011-11-062-0/+26
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bug is that an error would be raised while attempting to convert a template from one encoding to another. Please see http://redmine.ruby-lang.org/issues/5564 for more details. The workaround is to load all conversions into memory ahead of time, and will only happen if the ruby version is *exactly* 1.9.3p0. The hope is obviously that the underlying problem will be resolved in the next patchlevel release of 1.9.3.
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-053-5/+7
|\ \ \
| * | | Fix javascript_include_tag examples in documentationAlexey Vakhov2011-11-031-2/+2
| | | |
| * | | Fix small typos in routing docsAlexey Vakhov2011-11-031-1/+3
| | | |
| * | | Fix typo in Dispatcher#controller documentationAviv Ben-Yosef2011-11-011-1/+1
| | | |
| * | | Fix typo in constraints method documentationAlexey Vakhov2011-11-011-1/+1
| | | |
* | | | Remove 'size' attribute from number_field form helper fixes #3454Waseem Ahmad2011-11-041-0/+2
| |_|/ |/| | | | | | | | | | | f.number_field generates <input type="number", size="30"../> which is invalid HTML5. See: http://dev.w3.org/html5/spec/Overview.html#number-state
* | | this should never return nil, so no use for "try"Aaron Patterson2011-10-311-1/+1
| | |
* | | Ensure that the format isn't applied twice to the cache key, else it becomes ↵Christopher Meiklejohn2011-10-311-1/+1
|/ / | | | | | | impossible to target with expire_action.
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-10-302-4/+4
|\ \
| * | Adds missing closing regex slashes.mjy2011-10-251-2/+2
| | |
| * | add namespacing when referring to ActionController::RedirectBackErrorNick Howard2011-10-251-2/+2
| | |
* | | Add missing type to number_field_tag documentationPaul McMahon2011-10-281-1/+1
| | |
* | | scumbag 1.8: strings are enumerable :'(Aaron Patterson2011-10-261-1/+1
| | |
* | | Responders now return 204 No Content for API requests without a response ↵José Valim2011-10-261-21/+1
| | | | | | | | | | | | body (as in the new scaffold)
* | | properly handle lists of lists. Thanks @adrianpike for reporting!Aaron Patterson2011-10-251-1/+6
|/ /
* | Add ActionDispatch::Session::CacheStore as a generic way of storing sessions ↵Brian Durand2011-10-212-0/+51
| | | | | | | | in a cache.
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-10-202-3/+3
|\ \
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-28 07:57:25 +0000