| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | | Always use the provided port for protocol relative urls | Andrew White | 2014-05-11 | 1 | -0/+2 | |
| | | | | | | | | | | | There may be situations where you need to tunnel SSL connections over port 80 so we shouldn't remove it if it has been explicitly provided. | |||||
| * | | ActionDispatch::Http::URL.normalize_port should not strip port for protocol ↵ | Guilherme Cavalcanti | 2014-05-11 | 1 | -2/+0 | |
| | | | | | | | | | relative URL. | |||||
| * | | skip dealing with params if none are provided | Aaron Patterson | 2014-05-09 | 1 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | This lets us avoid 1. A slow call to Hash#slice 2. An is_a? test 3. Extra hash allocations (from slice) 4. String allocations etc. | |||||
| * | | don't mutate the options hash, so we don't have to dup | Aaron Patterson | 2014-05-09 | 1 | -4/+3 | |
| | | | | | | | | | avoids extra hash allocations on each call | |||||
| * | | use unless and || since these options are boolean | Aaron Patterson | 2014-05-09 | 1 | -1/+1 | |
| |/ | ||||||
| * | Refactor ActionDispatch::Http::URL.build_host_url | Andrew White | 2013-04-18 | 1 | -20/+49 | |
| | | | | | | Add support for extracting the port from the :host option and for removing the subdomain by using nil, false or ''. | |||||
| * | Duplicate options before mutating them | Andrew White | 2013-04-18 | 1 | -0/+1 | |
| | | ||||||
| * | Passing subdomain: '' to url_for removes the subdomain (instead of adding a ↵ | Derek Watson | 2013-04-18 | 1 | -1/+1 | |
| | | | | | | | leading .) Adding a boolean route constraint checks for presence/absence of request property | |||||
| * | Fix improperly configured host in generated urls | schneems | 2013-03-19 | 1 | -1/+12 | |
| | | | | | | | | | | | | | | | | | | | | If the host in `default_url_options` is accidentally set with a protocol such as ``` host: "http://example.com" ``` then the generated url will have the protocol twice `http://http://example.com` which is not what the user intended. Likely they wanted to define a host `host: "example.com"` and a `protocol: "http://"` but did not know the convention. This may not the most common problem, but when it happens it can go undetected for a while. I accidentally added `http://` out of habit recently only to find all the links in my emails were broken after deploying a demo site to production. Rather than allow this accident go undetected, we can fix the problem in line by properly setting the protocol and host. I was able to find this related question on stack overflow: http://stackoverflow.com/questions/5878329/rails-3-devise-how-do-i-make-the-email-confirmation-links-use-secure-https-n where the answer was highly upvoted. This is based off of work in #7415 cc/ @pixeltrix ATP Action Mailer and Action Pack | |||||
| * | Require AS deps in ActionDispatch::Http::URL | Miha Filej | 2013-02-19 | 1 | -0/+3 | |
| | | | | | Allows us to use url_for and friends in isolation. | |||||
| * | replace regexp global in #url_for | Yves Senn | 2013-01-16 | 1 | -2/+6 | |
| | | ||||||
| * | do not append a second slash when using | Yves Senn | 2013-01-02 | 1 | -1/+5 | |
| | | ||||||
| * | Correct the use of params options when given to url_for | Carlos Antonio da Silva | 2012-11-19 | 1 | -1/+1 | |
| | | | | | Merge url for tests and add changelog entry for #8233. | |||||
| * | Fix issue with params in url_for | tumayun | 2012-11-19 | 1 | -1/+1 | |
| | | | | | | | With a "params" argument, the following error is raised: undefined method `reject!` for "":String | |||||
| * | Refactor url methods a bit | Carlos Antonio da Silva | 2012-11-16 | 1 | -9/+9 | |
| | | | | | | | Use if..else conditions instead of return guards. Use _ for not used arguments when iterating. Set the path variable directly instead of using an empty string and <<. | |||||
| * | initialize instance variables | Aaron Patterson | 2012-08-09 | 1 | -0/+6 | |
| | | ||||||
| * | Remove unnecessary empty line | José Valim | 2012-04-25 | 1 | -1/+0 | |
| | | ||||||
| * | 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) | |||||
| * | Optimize path helpers. | José Valim | 2012-03-02 | 1 | -1/+3 | |
| | | ||||||
| * | Delegate subdomain method to the URL module, removing duplication | Carlos Antonio da Silva | 2012-01-12 | 1 | -1/+1 | |
| | | ||||||
| * | Simplify named host logic and missing host condition check | Carlos Antonio da Silva | 2012-01-12 | 1 | -2/+2 | |
| | | ||||||
| * | Extract ip host regexp to a constant | Carlos Antonio da Silva | 2012-01-12 | 1 | -1/+3 | |
| | | ||||||
| * | Fix url_for options[:subdomain] to allow objects as values | choonkeat | 2011-12-14 | 1 | -1/+1 | |
| | | | | | * e.g. blog_url(subdomain: current_user) instead of blog_url(subdomain: current_user.to_param) | |||||
| * | 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' | |||||
| * | / is allowed in URI fragments | Jeremy Kemper | 2011-10-13 | 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 | 1 | -3/+5 | |
| | | | | | for subdomain(s) removal from the host during link generation. Closes #2025 | |||||
| * | removing backwards compatibility module | Aaron Patterson | 2011-09-12 | 1 | -1/+1 | |
| | | ||||||
| * | minor correction to the ActionDispatch::Http::URL subdomain docs | Josh Kalderimis | 2011-05-04 | 1 | -1/+1 | |
| | | ||||||
| * | Fix .subdomain regression. | José Valim | 2011-05-04 | 1 | -1/+1 | |
| | | ||||||
| * | Filter params that return nil for to_param and allow through false values | Andrew White | 2011-03-09 | 1 | -1/+1 | |
| | | ||||||
| * | Protocol-relative URL support. | Stephen Celis | 2011-02-02 | 1 | -2/+5 | |
| | | | | | | | [#5774 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
| * | Wrap everything in class << self. | José Valim | 2010-12-03 | 1 | -42/+37 | |
| | | ||||||
| * | The redirect routing method now allows for a hash of options which only ↵ | Josh Kalderimis | 2010-11-30 | 1 | -0/+52 | |
| | | | | | changes the relevant parts of the url, or an object which responds to call can be supplied so common redirect rules can be easily reused. This commit includes a change where url generation from parts has been moved to AD::Http::URL as a class method. | |||||
| * | port_string bought back to life as it is part of the public api | Josh Kalderimis | 2010-11-24 | 1 | -3/+8 | |
| | | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
| * | :subdomain, :domain and :tld_length options can now be used in url_for, ↵ | Josh Kalderimis | 2010-11-23 | 1 | -20/+39 | |
| | | | | | | | allowing for easy manipulation of the host during link generation. Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
| * | Brought the domain method in AD http url inline with subdomain where ↵ | Josh Kalderimis | 2010-11-16 | 1 | -1/+2 | |
| | | | | | @@tld_length is used by default. Also set the default value of @@tld_length to 1. | |||||
| * | Fix loop introduced by rack:dda892d | Jeremy Kemper | 2010-10-26 | 1 | -6/+1 | |
| | | ||||||
| * | Rely on Rack::Session stores API for more compatibility across the Ruby world. | José Valim | 2010-10-03 | 1 | -5/+0 | |
| | | ||||||
| * | Cache 2 of Request's commonly called methods. | thedarkone | 2010-09-27 | 1 | -2/+2 | |
| | | ||||||
| * | Add configuration option for tld length | Simon Jefford | 2010-09-09 | 1 | -4/+6 | |
| | | ||||||
| * | Cleanup deprecations in Action Dispatch | Carlos Antonio da Silva | 2010-09-06 | 1 | -8/+1 | |
| | | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
| * | escape constants that should not be linked to | Joost Baaij | 2010-08-27 | 1 | -2/+2 | |
| | | ||||||
| * | Don't add the standard https port when using redirect in routes.rb and ↵ | Andrew White | 2010-08-20 | 1 | -0/+10 | |
| | | | | | | | | | ensure that request.scheme returns https when using a reverse proxy. [#5408 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
| * | Add caller to request_uri deprecation notice | Carlhuda | 2010-03-03 | 1 | -1/+1 | |
| | | ||||||
| * | ActionDispatch::Request deprecates #request_uri | Carl Lerche | 2010-03-03 | 1 | -10/+3 | |
| | | | | | * Refactored ActionPatch to use fullpath instead | |||||
| * | Rack::Request actually defines #query_string | Carl Lerche | 2010-03-03 | 1 | -5/+0 | |
| | | ||||||
| * | Tweak the semantic of various URL related methods of ActionDispatch::Request | Carlhuda | 2010-03-03 | 1 | -25/+6 | |
| | | ||||||
| * | Request#subdomain returns a string version of Request#subdomains | Joshua Peek | 2010-02-28 | 1 | -0/+4 | |
| | | ||||||
| * | Split ActionDispatch http in smaller chunks. | José Valim | 2010-01-16 | 1 | -0/+129 | |
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |