Coder-Xu777 opened a new issue, #12611: URL: https://github.com/apache/trafficserver/issues/12611
In trafficserver 9.2.x, I have some doubts about the logic of the following code snippet. https://github.com/apache/trafficserver/blob/9.2.x/proxy/http/HttpTransact.cc:3864 ``` if (s->api_server_addr_set) { // If the plugin set a server address, back up to the OS_DNS hook // to let it try another one. Force OS_ADDR_USE_CLIENT so that // in OSDNSLoopkup, we back up to how_to_open_connections which // will tell HttpSM to connect the origin server. s->dns_info.os_addr_style = DNSLookupInfo::OS_Addr::OS_ADDR_USE_CLIENT; TRANSACT_RETURN(SM_ACTION_API_OS_DNS, OSDNSLookup); } ``` The comments said we can try another one in OS_DNS hook, but there is no way to change the value of dns_info.lookup_success unless we add new api and it seems that none dns lookup operation could be performed during this process. Consequently, when OSDNSLookup is invoked, dns_info.lookup_success just keep false causing a failure ultimately. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
