shinrich commented on a change in pull request #7622:
URL: https://github.com/apache/trafficserver/pull/7622#discussion_r607152434
##########
File path: proxy/http/HttpTransact.cc
##########
@@ -1826,10 +1830,14 @@ HttpTransact::ReDNSRoundRobin(State *s)
s->next_action = how_to_open_connection(s);
} else {
// Our ReDNS failed so output the DNS failure error message
- build_error_response(s, HTTP_STATUS_BAD_GATEWAY, "Cannot find server.",
"connect#dns_failed");
+ // Set to internal server error so later logging will pick up
SQUID_LOG_ERR_DNS_FAIL
Review comment:
This is actually part of PR #7580 as is the line with your previous
comment. I'm hoping it lands first and will rebase out of this PR. Removing
it makes my rebase back to a prod build more sketchy. However, I've had no
luck getting anyone to look at it.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
