TS-2391: Traffic Server tries to reverse resolve 127.0.0.1
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/d22f9891 Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/d22f9891 Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/d22f9891 Branch: refs/heads/5.0.x Commit: d22f9891043bca75df2f44a36c7464d45f72936e Parents: f4e9fd6 Author: Bryan Call <[email protected]> Authored: Wed May 28 16:48:23 2014 -0700 Committer: Bryan Call <[email protected]> Committed: Wed May 28 16:48:23 2014 -0700 ---------------------------------------------------------------------- CHANGES | 2 ++ proxy/http/HttpTransact.cc | 6 ++---- 2 files changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/trafficserver/blob/d22f9891/CHANGES ---------------------------------------------------------------------- diff --git a/CHANGES b/CHANGES index 134d5cd..50ce778 100644 --- a/CHANGES +++ b/CHANGES @@ -1,6 +1,8 @@ -*- coding: utf-8 -*- Changes with Apache Traffic Server 5.0.0 + *) [TS-2391] Traffic Server tries to reverse resolve 127.0.0.1 + *) [TS-2837] Dangling pointer in URLImpl which may cause core dump *) [TS-2842] Can't set SPDY inactivity timeout with traffic_line http://git-wip-us.apache.org/repos/asf/trafficserver/blob/d22f9891/proxy/http/HttpTransact.cc ---------------------------------------------------------------------- diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc index ee59a8a..9efb233 100644 --- a/proxy/http/HttpTransact.cc +++ b/proxy/http/HttpTransact.cc @@ -1408,8 +1408,7 @@ HttpTransact::HandleRequest(State* s) if (s->dns_info.lookup_name[0] <= '9' && s->dns_info.lookup_name[0] >= '0' && (!s->state_machine->enable_redirection || !s->redirect_info.redirect_in_process) && - // (s->state_machine->authAdapter.needs_rev_dns() || - ( host_rule_in_CacheControlTable() || s->parent_params->ParentTable->hostMatch)) { + s->parent_params->ParentTable->hostMatch) { s->force_dns = 1; } //YTS Team, yamsat Plugin @@ -1834,8 +1833,7 @@ HttpTransact::OSDNSLookup(State* s) TRANSACT_RETURN(how_to_open_connection(s), HttpTransact::HandleResponse); } else if (s->dns_info.lookup_name[0] <= '9' && s->dns_info.lookup_name[0] >= '0' && - //(s->state_machine->authAdapter.needs_rev_dns() || - (host_rule_in_CacheControlTable() || s->parent_params->ParentTable->hostMatch) && + s->parent_params->ParentTable->hostMatch && !s->http_config_param->no_dns_forward_to_parent) { // note, broken logic: ACC fudges the OR stmt to always be true, // 'AuthHttpAdapter' should do the rev-dns if needed, not here .
