rawlinp commented on issue #4787: URL: https://github.com/apache/trafficcontrol/issues/4787#issuecomment-643324471
Oh, you tested the Python client? It is definitely broken by this change: ``` >>> a, _ = TO.create_server(data=s) ERROR:trafficops.restapi:restapi.py:_do_operation:424: POST request to RESTful API at [http://localhost:30003/api/2.0/servers] expected status(s) (200, 204); failed: 201 Created ``` I get that the API guidelines would like to move to 201s, but you can't do that without potentially breaking clients that depend on the previously-expected 200. Just adding 201 to the expected statuses without fixing this in the API is not good enough, unfortunately. ---------------------------------------------------------------- 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]
