]> git.koha-community.org Git - koha.git/commit
Bug 36901: Add logging for uncaught exceptions in background job classes
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 21 May 2024 13:04:17 +0000 (10:04 -0300)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Mon, 1 Jul 2024 16:55:54 +0000 (18:55 +0200)
commitb54c854d248f3c320f20d59224272d1631e27b0c
tree3a8641038f452f72ba342e5d61fae5567f1ae8d7
parent2e06546799ba815af805dfe1050dd459463e753a
Bug 36901: Add logging for uncaught exceptions in background job classes

This patch adds logging of unhandled exceptions that could occur. This
is happening on busy production sites right now. This is also useful for
plugin jobs that might not be 100% following the guidelines and would
benefit from this.

But as the [DO NOT PUSH] patch highlights, this is something we really
want to have on our current codebase, as a database connection drop
might make us reach that `catch` block we are adding logging to on this
patch.

To test:
1. Apply the [DO NOT PUSH] patch
2. Run:
   $ ktd --shell
  k$ restart_all ; tail -f /var/log/koha/kohadev/worker*.log
3. Pick a valid barcode on the staff UI
4. Use the 'Batch delete items' tool in the cataloguing section
5. Start the job for deleting the item
=> FAIL: The item got deleted, but the job marked as failed and no logs
about the reasons
6. Apply this patch and repeat 2-5
=> SUCCESS: Same scenario but there's a log with the error message
7. Sign off :-D

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
misc/workers/background_jobs_worker.pl