You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bc...@apache.org on 2014/05/29 01:48:56 UTC
git commit: TS-2391: Traffic Server tries to reverse resolve 127.0.0.1
Repository: trafficserver
Updated Branches:
refs/heads/master f4e9fd619 -> d22f98910
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/master
Commit: d22f9891043bca75df2f44a36c7464d45f72936e
Parents: f4e9fd6
Author: Bryan Call <bc...@apache.org>
Authored: Wed May 28 16:48:23 2014 -0700
Committer: Bryan Call <bc...@apache.org>
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 .