search

cessda / cessda.cvs.two

Apache License 2.0
0 stars 2 forks source link

Remove redundant catch blocks #802

Closed matthew-morris-cessda closed 4 months ago

matthew-morris-cessda commented 8 months ago

CVS has many try-catch blocks that only log that an error occured. These exceptions should be raised to a higher level (or possibly to the request layer) and handled there. This allows more meaningful debugging as well as better error messages for clients.

matthew-morris-cessda commented 4 months ago

Yes, the changes are covered by unit tests