[GitHub] [trafficcontrol] rob05c commented on issue #890: [TC-85] Should use "dest_ip" instead of "dest_domain" in "parent.config" and "cache.config" when ofqdn is ip

[GitBox <gi...@apache.org>]

rob05c commented on issue #890:
URL: https://github.com/apache/trafficcontrol/issues/890#issuecomment-658261719


   Yeah, it's still relevant. I don't think it's a huge issue, because we always use dest_domain, and MSO will put IPs in there, and it seems to work.
   
   It's still something we should fix, because ATS says it's wrong, so it could theoretically break in the future.
   
   We'll probably change TC config gen to always use Strategies, once we have them in ATS 9/10.
   
   But until then, I'd vote keep this issue open, just to not forget about 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:
users@infra.apache.org