From cf000685d4e2b1f2523b12eb84cb3a639b31578f Mon Sep 17 00:00:00 2001 From: David Cook Date: Fri, 20 Oct 2023 01:15:00 +0000 Subject: [PATCH] Bug 35111: Ignore SIGPIPE in background jobs worker Content-Type: text/plain; charset=utf-8 This change explicitly ignores SIGPIPE signals in the background jobs worker. Daemons like Starman ignore SIGPIPE so it makes sense to explicitly set this. Differences in the inner workings of MySQL vs MariaDB client libraries have yielded different behaviours in automatic reconnections and potentially SIGPIPE handling, so this helps to make the overall behaviour more consistent. Test plan: 0. Apply patch and run "restart_all" 1. Go to http://localhost:8081/cgi-bin/koha/catalogue/detail.pl?biblionumber=29 2. Click "Save" > "MARCXML" 3. Go to http://localhost:8081/cgi-bin/koha/tools/stage-marc-import.pl 4. Click "Choose file", choose the MARCXML file, click "Upload file" 5. Click "Stage for import" 6. Note the job is marked as "100% Finished" 7. In a separate window run "docker restart koha-db-1" 8. Repeat steps 3-5 for uploading file and running stage for import 9. Note that the job is marked as "100% Finished" as you'd expect Signed-off-by: Marcel de Rooy [EDIT] Added comment on the SIG PIPE line. --- misc/workers/background_jobs_worker.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/misc/workers/background_jobs_worker.pl b/misc/workers/background_jobs_worker.pl index 27ae8ff9be..fde3432143 100755 --- a/misc/workers/background_jobs_worker.pl +++ b/misc/workers/background_jobs_worker.pl @@ -64,6 +64,8 @@ use Koha::Logger; use Koha::BackgroundJobs; use C4::Context; +$SIG{'PIPE'} = 'IGNORE'; # See BZ 35111; added to ignore PIPE error when connection lost on Ubuntu. + my ( $help, @queues ); my $max_processes = $ENV{MAX_PROCESSES}; -- 2.30.2