Skip to content

Commit ad2e38a

Browse files
Dyrconasandbergja
authored andcommitted
LP2125232: Properly log SFTP errors in acquisitions EDI processing
The code for reporting SFTP get and put errors in acquistions EDI was not properly interpolating the error message into the log message. This patch resolves that by concatenating the value to the message string so that it appears properl in the logs and in the database. Release-note: Resolves an issue with some SFTP errors not being reported properly in acquisitions EDI processing. Signed-off-by: Jason Stephenson <[email protected]> Signed-off-by: Jane Sandberg <[email protected]>
1 parent 292e5a1 commit ad2e38a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Open-ILS/src/perlmods/lib/OpenILS/Utils/RemoteAccount.pm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ sub put_sftp {
389389
if ($self->_sftp->error or not $res) {
390390
$logger->error(
391391
$self->_error(
392-
"SFTP put to", $self->remote_host, "failed with error: $self->_sftp->error"
392+
"SFTP put to", $self->remote_host, "failed with error: " . $self->_sftp->error
393393
)
394394
);
395395
return;
@@ -412,7 +412,7 @@ sub get_sftp {
412412
if ($self->_sftp->error or not $success) {
413413
$logger->error(
414414
$self->_error(
415-
"get from", $self->remote_host, "failed with error: $self->_sftp->error"
415+
"get from", $self->remote_host, "failed with error: " . $self->_sftp->error
416416
)
417417
);
418418
return;

0 commit comments

Comments
 (0)