ocket8888 opened a new pull request #3996: Rewrote /user/current to Go
URL: https://github.com/apache/trafficcontrol/pull/3996
 
 
   ## What does this PR (Pull Request) do?
   - [x] This PR fixes #3838 
   
   This rewrites the previously unimplemented PUT handler of `/user/current` 
into Go. It adds support in both the Go and Python clients, with some 
associated testing as well as documentation updates.
   
   This differs from the way the endpoint was handled in Perl - in Perl, every 
field in the request was optional (under the top-level "user" object, 
obviously) and only those included would be updated - as though it were a PATCH 
request. This instead demands a full representation of a user as is proper for 
PUT. Additionally, the Perl returned only a success message, this handler 
returns a representation of the user as it exists following the update.
   
   ## Which Traffic Control components are affected by this PR?
   - Documentation
   - Traffic Control Client (Python)
   - Traffic Control Client (Go)
   - Traffic Ops
   
   ## What is the best way to verify this PR?
   Build a TO RPM, start Traffic Ops, make a `PUT` request to `/user/current`, 
run the Go client/API integration tests, build and read the documentation.
   
   ## The following criteria are ALL met by this PR
   - [x] This PR includes tests
   - [x] This PR includes documentation
   - [x] This PR includes an update to CHANGELOG.md
   - [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]


With regards,
Apache Git Services

Reply via email to