Bug 39911 - Fatal errors from SIP server are not logged
Summary: Fatal errors from SIP server are not logged
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: SIP2 (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low major
Assignee: Kyle M Hall (khall)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-15 17:23 UTC by Kyle M Hall (khall)
Modified: 2025-05-16 07:01 UTC (History)
3 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 39911: Introduce failure in patron information request [DO NOT PUSH] (746 bytes, patch)
2025-05-15 17:30 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 39911: Trap DIE signals in SIP server (1.43 KB, patch)
2025-05-15 17:30 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 39911: Trap DIE signals in SIP server (1.47 KB, patch)
2025-05-15 17:31 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 39911: Trap DIE signals in SIP server (1.51 KB, patch)
2025-05-16 07:01 UTC, Magnus Enger
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 2025-05-15 17:23:10 UTC
At some point our SIP server stopped logging fatal errors to either logs or standard error. We should restore this vital functionality.
Comment 1 Kyle M Hall (khall) 2025-05-15 17:30:01 UTC
Created attachment 182501 [details] [review]
Bug 39911: Introduce failure in patron information request [DO NOT PUSH]
Comment 2 Kyle M Hall (khall) 2025-05-15 17:30:23 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall (khall) 2025-05-15 17:31:53 UTC Comment hidden (obsolete)
Comment 4 Magnus Enger 2025-05-16 07:01:36 UTC
Created attachment 182508 [details] [review]
Bug 39911: Trap DIE signals in SIP server

At some point our SIP server stopped logging fatal errors to either logs or standard error. We should restore this vital functionality.

Test Plan:
1) Apply the first patch which will cause the patron information request
   to fail
2) Stop the SIP server
3) Start it again with: perl /kohadevbox/koha/C4/SIP/SIPServer.pm /etc/koha/sites/kohadev/SIPconfig.xml
4) Trigger the failure with the following command:
   ./misc/sip_cli_emulator.pl -a 127.0.0.1 -p 6001 -su koha -sp koha -l MPL --patron koha -m patron_information
5) Note "THIS IS A TEST" is not found in logs or stderr
6) Apply the second patch
7) Repeat steps 2-4
8) Note the error message is now visible from stderr at the command line
   and also in the SIP log

Signed-off-by: Magnus Enger <magnus@libriotech.no>