]> git.koha-community.org Git - koha.git/commit
Bug 36421: Make sure pre-controller errors are correctly logged
authorTomas Cohen Arazi <tomascohen@theke.io>
Wed, 27 Mar 2024 12:29:31 +0000 (12:29 +0000)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Thu, 4 Apr 2024 13:24:29 +0000 (15:24 +0200)
commite518e0be2c97782997037383223e57ce2e196df4
tree11eb573ef43f7f6e6039fc61b722824b897007a6
parent3ed4e096b017a539a3b9afdb913f9d61fb43a0f5
Bug 36421: Make sure pre-controller errors are correctly logged

If something explodes in Koha::REST::V1::Auth chances are that no logging will happen.

This patch makes sure they are!

To test:
1. Make sure you do NOT have 36420 on your tree
2. Make sure you are looking at the logs:
   $ ktd --shell
  k$ koha-plack --restart kohadev; tail -f /var/log/koha/kohadev/*.log
3. Use Postman or similar for hitting some known endpoint. Use
   the user's cardnumber instead of the userid. On a default KTD launch, the
   generated user's cardnumber is '42'.
   GET /patrons
=> FAIL: You get a 500 error (expected, fixed on 36420) but no useful logging found.
4. Apply this patch
5. Ctrl+c on the logs and re-run the command
6. Repeat 3
=> SUCCESS: You get a 500, but you also get the exception information logged!
7. Sign off :-D

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
[OK] prove t/db_dependent/api/v1/items.t t/db_dependent/api/v1/unhandled_exceptions.t
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Koha/REST/Plugin/Exceptions.pm
Koha/REST/V1/Auth.pm