[GitHub] [trafficcontrol] ocket8888 commented on issue #4232: 500 Internal Server Error on clicking Generate ISO from ISO page

[GitBox <gi...@apache.org>]

ocket8888 commented on issue #4232: 500 Internal Server Error on clicking Generate ISO from ISO page
URL: https://github.com/apache/trafficcontrol/issues/4232#issuecomment-570026242
 
 
   > _"So maybe this is not a bug as it works fine IF everything if the enviro is setup properly? So maybe this is an API improvement instead?"_
   
   Agree. But I believe our standard behavior for bad/missing configuration causing an API endpoint to not function (e.g. no available InfluxDB servers for stats requests, no configured mailserver etc.) is to not expose that information through the API. That wasn't a mailing list decision afaik but it was pointed out in comments of one or two PRs in the past (that I'm aware of). So according to that philosophy we should just log the problem and hand back a `500 Internal Server Error` to the client, I think.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services