ocket8888 commented on a change in pull request #3954: Added deprecation
notices for /user/current/update
URL: https://github.com/apache/trafficcontrol/pull/3954#discussion_r333111865
##########
File path: traffic_ops/app/lib/API/User.pm
##########
@@ -599,6 +599,123 @@ sub current {
}
}
+# Designated handler for the deprecated path to updating current users
+sub user_current_update {
Review comment:
The current handler is also the handler for `PUT /user/current`, which isn't
deprecated. Since the rendering happens before the handler returns, I couldn't
just wrap it, and if there's a way to access what path was requested from
within a Mojolicious handler then I don't know it. So this seemed to be my only
option.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
