This is an automated email from the ASF dual-hosted git repository.

rrm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new a717eb721 fix: ensure DNS resolution for parent proxy (#9054)
a717eb721 is described below

commit a717eb7217818df97c96fb8a3e1619ffdd65b85d
Author: fatih-acar <fatih-a...@users.noreply.github.com>
AuthorDate: Fri Sep 23 19:22:06 2022 +0200

    fix: ensure DNS resolution for parent proxy (#9054)
    
    HostDB restructure introduced a regression where DNS resolution for
    parent proxies would be skipped and the origin's address would be used
    instead.
    
    Fixes: #8953
    
    Signed-off-by: Fatih Acar <fa...@scaleway.com>
    Co-authored-by: Fatih Acar <fa...@scaleway.com>
---
 proxy/http/HttpSM.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 5b9699390..e349181bf 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -7571,7 +7571,7 @@ HttpSM::set_next_state()
       t_state.dns_info.resolved_p = true; // seems dangerous - where's the IP 
address?
       call_transact_and_set_next_state(nullptr);
       break;
-    } else if (t_state.dns_info.resolve_immediate()) {
+    } else if (t_state.parent_result.result == PARENT_UNDEFINED && 
t_state.dns_info.resolve_immediate()) {
       call_transact_and_set_next_state(nullptr);
       break;
     }

Reply via email to