search

apache / trafficcontrol

Apache Traffic Control is an Open Source implementation of a Content Delivery Network
https://trafficcontrol.apache.org/
Apache License 2.0
1.06k stars 344 forks source link

Remove CacheURL Delivery Service Field #3225

Open jhg03a opened 5 years ago

jhg03a commented 5 years ago

CacheURL was a plugin that was deprecated in ATS6 and removed in ATS7. ATS6 is now dead. We should remove all traces of cacheURL from the TO API, TODB Database, Native-client API Libraries, Documentation, & TP Web UI.

This is previously discussed in #2957 and #1130

jhg03a commented 5 years ago

Consensus appears to have been reached and a deprecation notice issued for the 3.x release.

https://lists.apache.org/thread.html/d0a8805315819b5e4d9d9bf7f19bfd2b895924dc4247bec299b3ba15@%3Cdev.trafficcontrol.apache.org%3E

shamrickus commented 3 years ago

Working on this, see #4644

mitchell852 commented 3 years ago

i feel like this should be done by now with 4 merged PRs :)

is it @shamrickus ?

shamrickus commented 3 years ago

No. It's now officially deprecated, which it wasn't before. We were returning the field in api v3 so until that's gone we can't get rid of it completely

mitchell852 commented 3 years ago

No. It's now officially deprecated, which it wasn't before. We were returning the field in api v3 so until that's gone we can't get rid of it completely

gotcha. thanks for the clarification. hard to get rid of these things, huh?

mitchell852 commented 3 years ago

TC 5.x and 6.x supports ATS 7.x+, right? maybe i'm misunderstanding this but if the cacheUrl field exists in the 1.x, 2.x and 3.x versions of TC (1.x soon to be removed) but it's only used in ATS 6.x-, can't we call this a minor issue as nothing really uses this value?