ocket8888 commented on pull request #5162: URL: https://github.com/apache/trafficcontrol/pull/5162#issuecomment-710741987
> Maybe context would only be added if it was a SysErr, in which case you could use a helper function that only adds context if it's a SysErr. Yeah that sounds great. I was actually considering doing something similar to that in the current version by the time I was done seeing all the use-cases. > ...are there any cases where we actually set both errors? Yes. I think there's only two or three, but here's one: [lines 174 through 178 in login/register.go](https://github.com/apache/trafficcontrol/blob/master/traffic_ops/traffic_ops_golang/login/register.go#L174-L178) ---------------------------------------------------------------- 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]
