ocket8888 commented on a change in pull request #4142: Allow IPv6 Only Caches, including monitoring and routing URL: https://github.com/apache/trafficcontrol/pull/4142#discussion_r379037787
########## File path: traffic_monitor/manager/manager.go ########## @@ -45,6 +46,8 @@ import ( func Start(opsConfigFile string, cfg config.Config, appData config.StaticAppData, trafficMonitorConfigFileName string) error { toSession := towrap.ITrafficOpsSession(towrap.NewTrafficOpsSessionThreadsafe(nil, cfg.CRConfigHistoryCount, cfg)) + url6Regex := regexp.MustCompile(`/\d+`) Review comment: Sure but if the result of stripping is used for TCP dials then you're paying that cost anyway. Idk how it's used later on, just wanted to throw that out there. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services