[GitHub] [trafficcontrol] zrhoffman opened a new issue, #7801: API v5 client.CreateDeliveryServicesRequiredCapability 404s

["zrhoffman (via GitHub)" <gi...@apache.org>]

zrhoffman opened a new issue, #7801:
URL: https://github.com/apache/trafficcontrol/issues/7801

   <!--
   ************ STOP!! ************
   If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact
   the Apache Traffic Control Security Team at security@trafficcontrol.apache.org and follow the
   guidelines at https://apache.org/security regarding vulnerability disclosure.
   
   - For *SUPPORT QUESTIONS*, use the #traffic-control channel on the ASF slack (https://s.apache.org/tc-slack-request)
   or the Traffic Control Users mailing list (send an email to users-subscribe@trafficcontrol.apache.org to subscribe).
   - Before submitting, please **SEARCH GITHUB** for a similar issue or PR
       * https://github.com/apache/trafficcontrol/issues
       * https://github.com/apache/trafficcontrol/pulls
   -->
   
   <!-- Do not submit security vulnerabilities or support requests here - see above -->
   ## This Bug Report affects these Traffic Control components:
   <!-- delete all those that don't apply -->
   - Traffic Control Cache Config (`t3c`, formerly ORT)
   - Traffic Control Client - Go library
   - Traffic Ops - v5 API tests
   - CDN in a Box - enroller, Traffic Ops data
   
   ## What did you do?
   <!-- If the current behavior is a bug, please provide the *STEPS TO REPRODUCE* and
   include the applicable TC version.
   -->
   The enroller tried to create a DS required capability using `client.CreateDeliveryServicesRequiredCapability` (in API v5)
   
   ## What did you expect to happen?
   <!-- Describe how the bug happens -->
   Capability is created
   
   ## What actually happened?
   <!-- Describe what the behavior would be without the bug -->
   ```
   error creating Delivery Services Required Capability: error requesting Traffic Ops: path 'https://trafficops.infra.ciab.test:443/api/5.0/deliveryservices_required_capabilities' gave HTTP error 404 Not Found - error-level alerts: The requested path '/api/5.0/deliveryservices_required_capabilities' does not exist.
   ```
   
   It was removed in API v5, so these client methods should be removed:
   `CreateDeliveryServicesRequiredCapability`
   `DeleteDeliveryServicesRequiredCapability`
   `GetDeliveryServicesRequiredCapabilities`
   
   And the places using them should be updated use Delivery Services directly.
   
   ```
   cache-config/t3cutil/toreq/clientfuncs.go
   cache-config/t3cutil/toreq/toreqold/clientfuncs.go
   infrastructure/cdn-in-a-box/enroller/enroller.go
   lib/go-tc/totest/deliveryservices_required_capabilities.go
   traffic_ops/testing/api/v5/deliveryservices_required_capabilities_test.go
   traffic_ops/traffic_ops_golang/deliveryservice/deliveryservices_required_capabilities_test.go
   ```


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

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] rimashah25 closed issue #7801: API v5 client.CreateDeliveryServicesRequiredCapability 404s

["rimashah25 (via GitHub)" <gi...@apache.org>]

rimashah25 closed issue #7801: API v5 client.CreateDeliveryServicesRequiredCapability 404s
URL: https://github.com/apache/trafficcontrol/issues/7801


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

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org