@diosmosis opened this Pull Request on January 25th 2019 Member

Added two new exceptions HttpCodeException and BadRequestException that allows changing the HTTP code returned for an exception (in both controller & API).

This allows us to avoid certain failures caused by invalid input and correctly report the error, w/o triggering a 500 HTTP code (that could potentially also create an email alert?).

Used in ScheduledReports API to check for multiple periods.

Fixes #13752

@sgiehl commented on February 23rd 2019 Member

@diosmosis this one needs a rebase / merge to solve the conflicts

@tsteur commented on April 11th 2019 Member

Left 2 comments, otherwise good to merge when tests pass 👍

This Pull Request was closed on May 15th 2019
Powered by GitHub Issue Mirror