zrhoffman commented on issue #4946:
URL: https://github.com/apache/trafficcontrol/issues/4946#issuecomment-680182837


   > > I'm not sure why we're manipulating responses within the TO client 
itself, but we've done that since our initial commit, so it isn't anything new.
   > 
   > Because it's more convenient to work with.
   > 
   > Why didn't the TO endpoint just return the more convenient structure? 
Because the person who wrote it wasn't the person who had to use it.
   
   Oh, I didn't mean as opposed to TO returning a more convenient structure, I 
meant as opposed to the change I'm proposing, having TM call 
`client.GetTrafficMonitorConfig()` directly and then later calling 
`tc.LegacyTrafficMonitorTransformToMap()` itself so operators can just copy the 
response from `GET /cdns/{{CDN}}/configs/monitoring` to `tmconfig.backup` 
without TM having needed to have an initial connection to TO and generate the 
backup file itself, as @ocket8888 said.


----------------------------------------------------------------
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]


Reply via email to