ocket8888 opened a new pull request #4794:
URL: https://github.com/apache/trafficcontrol/pull/4794


   ## What does this PR (Pull Request) do?
   
   - [x] This PR fixes is not related to any issue
   
   Fixes changes to the v1/v2 API which modified the `POST /servers` response 
code from `200 OK` to `201 Created`, and also changes some alert messages in 
those versions back to their old content.
   
   Also changes the TO Python client to no longer incorrectly reject successful 
responses as failures.
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Control Client (Python)
   - Traffic Ops
   
   ## What is the best way to verify this PR?
   Create servers in v1/v2 - probably using the Python client to kill two birds 
with one stone - and ensure that the response code is 201. Also maybe verify 
the alert messages, if you care - nothing should depend on those exact words, 
but I changed them back just because.
   
   ## If this is a bug fix, what versions of Traffic Control are affected?
   - master
   
   ## The following criteria are ALL met by this PR
   - [x] Tests are unnecessary
   - [x] Documentation is unnecessary
   - [x] An update to CHANGELOG.md is not necessary
   - [x] This PR includes any and all required license headers
   - [x] This PR does not include a database migration
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY**


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