You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by br...@apache.org on 2015/02/04 23:55:31 UTC

[1/2] trafficserver git commit: TS-3366: TS shouldn't use HostDB for 127.0.0.1 or ::1

Repository: trafficserver
Updated Branches:
  refs/heads/master be357bb1c -> 0509bf64c


TS-3366: TS shouldn't use HostDB for 127.0.0.1 or ::1


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/6f03db28
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/6f03db28
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/6f03db28

Branch: refs/heads/master
Commit: 6f03db28de149da0bc85cf2b4ff3b878e7de5011
Parents: be357bb
Author: Brian Geffon <br...@apache.org>
Authored: Wed Feb 4 14:55:02 2015 -0800
Committer: Brian Geffon <br...@apache.org>
Committed: Wed Feb 4 14:55:02 2015 -0800

----------------------------------------------------------------------
 proxy/http/HttpSM.cc | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/6f03db28/proxy/http/HttpSM.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 28990de..34d0d05 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -7039,7 +7039,14 @@ HttpSM::set_next_state()
     {
       sockaddr const* addr;
 
-      if (t_state.api_server_addr_set) {
+      if ((strncmp(t_state.dns_info.lookup_name, "127.0.0.1", 9) == 0 || strncmp(t_state.dns_info.lookup_name, "::1",3) == 0)
+          && ats_ip_pton(t_state.dns_info.lookup_name, t_state.host_db_info.ip()) == 0) {
+        // If it's 127.0.0.1 or ::1 don't bother with hostdb
+        DebugSM("dns", "[HttpTransact::HandleRequest] Skipping DNS lookup for %s because it's loopback", t_state.dns_info.lookup_name);
+        t_state.dns_info.lookup_success = true;
+        call_transact_and_set_next_state(NULL);
+        break;
+      } else if (t_state.api_server_addr_set) {
         /* If the API has set the server address before the OS DNS lookup
          * then we can skip the lookup
          */


[2/2] trafficserver git commit: TS-3366: update changes

Posted by br...@apache.org.
TS-3366: update changes


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/0509bf64
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/0509bf64
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/0509bf64

Branch: refs/heads/master
Commit: 0509bf64cdf6ead38de20b2227bea8d8691545e7
Parents: 6f03db2
Author: Brian Geffon <br...@apache.org>
Authored: Wed Feb 4 14:55:25 2015 -0800
Committer: Brian Geffon <br...@apache.org>
Committed: Wed Feb 4 14:55:25 2015 -0800

----------------------------------------------------------------------
 CHANGES | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/0509bf64/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index bc2e794..8cd3e91 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache Traffic Server 5.3.0
 
+  *) [TS-3366] TS shouldn't use HostDB for 127.0.0.1 or ::1
+
   *) [TS-2480] Fix session ticket key lookup to work with the default context bootstrap
 
   *) [TS-3299] Fix leaking vc's during inactivitycop, due to null continuation