You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/06/07 21:36:10 UTC

[GitHub] [trafficcontrol] ocket8888 opened a new issue #5921: Remove the /deliveryservices/request TO API endpoint

ocket8888 opened a new issue #5921:
URL: https://github.com/apache/trafficcontrol/issues/5921


   ## I'm submitting a ...
   -  other
   
   ## Traffic Control components affected ...
   -  Traffic Ops
   
   ## Current behavior:
   The `/deliveryservices/request` endpoint exists
   
   ## Expected behavior:
   It doesn't existt
   
   ## Minimal reproduction of the problem with instructions:
   Make a request to `/deliveryservices/request` at the latest API version, notice that a 404 response is not returned.
   
   ## Anything else:
   This endpoint was used internally at Comcast as a part of a process that is no longer used. It represents a point of confusion with actual "Delivery Service Requests" and some additional testing and usage complexity.


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

[GitHub] [trafficcontrol] TaylorCFrey commented on issue #5921: Remove the /deliveryservices/request TO API endpoint

Posted by GitBox <gi...@apache.org>.

TaylorCFrey commented on issue #5921:
URL: https://github.com/apache/trafficcontrol/issues/5921#issuecomment-856833662


   I'm in favor!


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

[GitHub] [trafficcontrol] rawlinp commented on issue #5921: Remove the /deliveryservices/request TO API endpoint

Posted by GitBox <gi...@apache.org>.

rawlinp commented on issue #5921:
URL: https://github.com/apache/trafficcontrol/issues/5921#issuecomment-856868827


   I'm also in favor, and I think it would be fine to remove the routes without doing the `410 Gone` response or adding deprecation notices to earlier versions. As long as we have an entry in the `Removed` section of the changelog, I think that's fine.


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

[GitHub] [trafficcontrol] ocket8888 commented on issue #5921: Remove the /deliveryservices/request TO API endpoint

Posted by GitBox <gi...@apache.org>.

ocket8888 commented on issue #5921:
URL: https://github.com/apache/trafficcontrol/issues/5921#issuecomment-856859297


   Regarding implementation: we could strip the functionality from all API versions at once, because that reduces the complexity immediately, and then older versions where the endpoint used to exist can return a `410 Gone` response instead of a 404. Either that, or remove it from API v4 and add deprecation notices to earlier versions.


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