https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25032
--- Comment #16 from Tomás Cohen Arazi <tomasco...@gmail.com> --- Created attachment 102524 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102524&action=edit Bug 25032: Generic unhandled exception handling on API This patch adds Koha::Logger as the default logger for the API, and introduces a new helper plugin that takes care of handling the unhandled exceptions. Basically, with this we would write something like this in our controller methods: try { ... } catch { if ( know_exception ) { handle_known_exception($_); } $c->unhandled_exception($_); } Without this, we end up adding more and more handling 'just in case'. To test: 1. Edit the Koha/REST/V1/Cities.pm 'list' method adding die("Nada"); before the render step. 2. Restart plack and try the endpoint => FAIL: A generic error is displayed, and no traces of the original problem are found on the logs. 3. Apply this patches, make sure your instance's log4perl has the introduced lines for API with the right path. 4. Repeat 2 => SUCCESS: The message is still generic, but you see something is logged in /var/log/koha/kohadev/api-error.log 5. Change die("Nada"); for a real exception like: use Koha::Exceptions; Koha::Exceptions::DuplicateObject->throw("Nada"); 6. Repeat 2. => SUCCESS: The message is generic, but a meaningful text is added to the logs. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/