https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42993
Bug ID: 42993
Summary: Koha::Edifact::Transport should log transport failures
via Koha::Logger
Initiative type: ---
Sponsorship ---
status:
Product: Koha
Version: Main
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: [email protected]
Reporter: [email protected]
QA Contact: [email protected]
Target Milestone: ---
Koha::Edifact::Transport reports every transport failure (no transport
configured, connect failure, directory-change failure, file-listing
failure and per-file download/upload failure) with carp. Under
misc/cronjobs/edi_cron.pl these go to STDERR, so on a normal cron run
the messages are lost: EDI simply goes quiet with nothing in
editrace.log to explain why.
This routes those failures through Koha::Logger on the 'edi' interface
(the same interface edi_cron.pl already uses) so they are recorded in
the EDI log. The two ingest()-level data warnings (duplicate filename,
unreadable file) are intentionally left as carp: one is asserted by the
existing unit test and neither is a transport failure.
No change to the transfer logic.
Test plan:
1. prove t/db_dependent/Koha/Edifact/Transport.t
2. Point an EDI account at an unreachable transport and run
misc/cronjobs/edi_cron.pl; confirm the failure is now written to the
EDI log rather than only to STDERR.
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
