[Bug 40840] New: Failure of an EDI SFTP account will kill edi_cron.pl preventing uploads from subsequent accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Bug ID: 40840 Summary: Failure of an EDI SFTP account will kill edi_cron.pl preventing uploads from subsequent accounts Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org EDI uploads are processed in order. If one account is unable to connect to the SFTP server, it will call die_on_error which kills the script. This does not affect regular FTP which will fail gracefully. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|major |normal -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 --- Comment #1 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- Created attachment 186574 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186574&action=edit Bug 40840: Failure of an EDI SFTP account will kill edi_cron.pl preventing uploads from subsequent accounts EDI uploads are processed in order. If one account is unable to connect to the SFTP server, it will call die_on_error which kills the script. This does not affect regular FTP which will fail gracefully. Test Plan: 1) Set up an SFTP server 2) Set up 3 EDI accounts 3) Set accounts 1 and 3 to have a good SFTP server hostname 4) Set account 2 to have a bad SFTP server hostname 5) Queue an edi order for each account 6) Run edi_cron.pl 7) Note the cronjob runs and dies with an error 8) Note only 1 EDI message was uploaded 9) Apply this patch 10) Run edi_cron.pl 11) Note that the cronjob does not die 12) Verify the message for account 3 was uploaded! ( the message for account 2 should remain as pending ). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Kyle M Hall (khall) <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Does someone have an SFTP server/account that could be used to test this? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 --- Comment #3 from Kyle M Hall (khall) <kyle@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #2)
Does someone have an SFTP server/account that could be used to test this?
I believe you can just set up SSH on your KTD and use that: sudo apt install openssh-server sudo passwd kohadev-koha sftp kohadev-koha@localhost -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@openfifth.c |y.org |o.uk CC| |lisette@bywatersolutions.co | |m -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |25.05 --- Comment #4 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- This is only required for 25.05 at this stage as the transport system was reworked for 25.11 and beyond. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #186574|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 --- Comment #5 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 200495 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200495&action=edit Bug 40840: Failure of an EDI SFTP account will kill edi_cron.pl preventing uploads from subsequent accounts EDI uploads are processed in order. If one account is unable to connect to the SFTP server, it will call die_on_error which kills the script. This does not affect regular FTP which will fail gracefully. Test Plan: 1) Set up an SFTP server 2) Set up 3 EDI accounts 3) Set accounts 1 and 3 to have a good SFTP server hostname 4) Set account 2 to have a bad SFTP server hostname 5) Queue an edi order for each account 6) Run edi_cron.pl 7) Note the cronjob runs and dies with an error 8) Note only 1 EDI message was uploaded 9) Apply this patch 10) Run edi_cron.pl 11) Note that the cronjob does not die 12) Verify the message for account 3 was uploaded! ( the message for account 2 should remain as pending ). Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Martin Renvoize (ashimema) from comment #4)
This is only required for 25.05 at this stage as the transport system was reworked for 25.11 and beyond.
Should this have a rel-keyword for 25.05 then? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40840 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au Keywords| |rel_25_05_candidate --- Comment #7 from David Cook <dcook@prosentient.com.au> --- (In reply to Katrin Fischer from comment #6)
(In reply to Martin Renvoize (ashimema) from comment #4)
This is only required for 25.05 at this stage as the transport system was reworked for 25.11 and beyond.
Should this have a rel-keyword for 25.05 then?
I think so. Adding now. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org