vh/termscp
1
0
Fork 0
mirror of https://github.com/veeso/termscp.git synced 2025-12-07 09:36:00 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

File transfer errors: to_string instead of format!

Browse Source
This commit is contained in:
veeso
2021-04-04 16:32:54 +02:00
parent 47cd112e69
commit f136057484
3 changed files with 53 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
src/filetransfer/ftp_transfer.rs
View File

@@ -329,7 +329,7 @@ impl FileTransfer for FtpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -344,7 +344,7 @@ impl FileTransfer for FtpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::SslError, FileTransferErrorType::SslError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -353,7 +353,7 @@ impl FileTransfer for FtpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::SslError, FileTransferErrorType::SslError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -396,7 +396,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -422,7 +422,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(path) => Ok(PathBuf::from(path.as_str())), Ok(path) => Ok(PathBuf::from(path.as_str())),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -442,7 +442,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(dir), Ok(_) => Ok(dir),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -482,7 +482,7 @@ impl FileTransfer for FtpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::DirStatFailed, FileTransferErrorType::DirStatFailed,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -501,7 +501,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -527,7 +527,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::PexError, FileTransferErrorType::PexError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -540,7 +540,7 @@ impl FileTransfer for FtpFileTransfer {
if let Err(err) = self.remove(&file) { if let Err(err) = self.remove(&file) {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::PexError, FileTransferErrorType::PexError,
format!("{}", err), err.to_string(),
)); ));
} }
} }
@@ -549,13 +549,13 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::PexError, FileTransferErrorType::PexError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::DirStatFailed, FileTransferErrorType::DirStatFailed,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -588,7 +588,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -638,7 +638,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(writer) => Ok(Box::new(writer)), // NOTE: don't use BufWriter here, since already returned by the library Ok(writer) => Ok(Box::new(writer)), // NOTE: don't use BufWriter here, since already returned by the library
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -657,7 +657,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(reader) => Ok(Box::new(reader)), // NOTE: don't use BufReader here, since already returned by the library Ok(reader) => Ok(Box::new(reader)), // NOTE: don't use BufReader here, since already returned by the library
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -679,7 +679,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(
@@ -701,7 +701,7 @@ impl FileTransfer for FtpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
}, },
None => Err(FileTransferError::new( None => Err(FileTransferError::new(

32
src/filetransfer/scp_transfer.rs
View File

@@ -315,7 +315,7 @@ impl FileTransfer for ScpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::BadAddress, FileTransferErrorType::BadAddress,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -346,7 +346,7 @@ impl FileTransfer for ScpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -356,7 +356,7 @@ impl FileTransfer for ScpFileTransfer {
if let Err(err) = session.handshake() { if let Err(err) = session.handshake() {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)); ));
} }
let username: String = match username { let username: String = match username {
@@ -378,7 +378,7 @@ impl FileTransfer for ScpFileTransfer {
) { ) {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::AuthenticationFailed, FileTransferErrorType::AuthenticationFailed,
format!("{}", err), err.to_string(),
)); ));
} }
} }
@@ -390,7 +390,7 @@ impl FileTransfer for ScpFileTransfer {
) { ) {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::AuthenticationFailed, FileTransferErrorType::AuthenticationFailed,
format!("{}", err), err.to_string(),
)); ));
} }
} }
@@ -422,7 +422,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -492,7 +492,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -534,7 +534,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -573,7 +573,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -610,7 +610,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -647,7 +647,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -689,7 +689,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -744,7 +744,7 @@ impl FileTransfer for ScpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -765,7 +765,7 @@ impl FileTransfer for ScpFileTransfer {
Ok(output) => Ok(output), Ok(output) => Ok(output),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -824,7 +824,7 @@ impl FileTransfer for ScpFileTransfer {
Ok(channel) => Ok(Box::new(BufWriter::with_capacity(65536, channel))), Ok(channel) => Ok(Box::new(BufWriter::with_capacity(65536, channel))),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -847,7 +847,7 @@ impl FileTransfer for ScpFileTransfer {
Ok(reader) => Ok(Box::new(BufReader::with_capacity(65536, reader.0))), Ok(reader) => Ok(Box::new(BufReader::with_capacity(65536, reader.0))),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
} }
} }

40
src/filetransfer/sftp_transfer.rs
View File

@@ -76,12 +76,12 @@ impl SftpFileTransfer {
Ok(_) => Ok(p), Ok(_) => Ok(p),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
}, },
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -90,7 +90,7 @@ impl SftpFileTransfer {
Ok(_) => Ok(p), Ok(_) => Ok(p),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
}, },
Err(_) => Err(FileTransferError::new( Err(_) => Err(FileTransferError::new(
@@ -260,7 +260,7 @@ impl FileTransfer for SftpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::BadAddress, FileTransferErrorType::BadAddress,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -291,7 +291,7 @@ impl FileTransfer for SftpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -301,7 +301,7 @@ impl FileTransfer for SftpFileTransfer {
if let Err(err) = session.handshake() { if let Err(err) = session.handshake() {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)); ));
} }
let username: String = match username { let username: String = match username {
@@ -323,7 +323,7 @@ impl FileTransfer for SftpFileTransfer {
) { ) {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::AuthenticationFailed, FileTransferErrorType::AuthenticationFailed,
format!("{}", err), err.to_string(),
)); ));
} }
} }
@@ -335,7 +335,7 @@ impl FileTransfer for SftpFileTransfer {
) { ) {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::AuthenticationFailed, FileTransferErrorType::AuthenticationFailed,
format!("{}", err), err.to_string(),
)); ));
} }
} }
@@ -348,7 +348,7 @@ impl FileTransfer for SftpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -358,7 +358,7 @@ impl FileTransfer for SftpFileTransfer {
Err(err) => { Err(err) => {
return Err(FileTransferError::new_ex( return Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)) ))
} }
}; };
@@ -386,7 +386,7 @@ impl FileTransfer for SftpFileTransfer {
} }
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ConnectionError, FileTransferErrorType::ConnectionError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -459,7 +459,7 @@ impl FileTransfer for SftpFileTransfer {
match sftp.readdir(dir.as_path()) { match sftp.readdir(dir.as_path()) {
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::DirStatFailed, FileTransferErrorType::DirStatFailed,
format!("{}", err), err.to_string(),
)), )),
Ok(files) => { Ok(files) => {
// Allocate vector // Allocate vector
@@ -490,7 +490,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -517,7 +517,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::PexError, FileTransferErrorType::PexError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -538,7 +538,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::PexError, FileTransferErrorType::PexError,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -562,7 +562,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(_) => Ok(()), Ok(_) => Ok(()),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -585,7 +585,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(metadata) => Ok(self.make_fsentry(dir.as_path(), &metadata)), Ok(metadata) => Ok(self.make_fsentry(dir.as_path(), &metadata)),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -604,7 +604,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(output) => Ok(output), Ok(output) => Ok(output),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::ProtocolError, FileTransferErrorType::ProtocolError,
format!("{}", err), err.to_string(),
)), )),
}, },
false => Err(FileTransferError::new( false => Err(FileTransferError::new(
@@ -643,7 +643,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(file) => Ok(Box::new(BufWriter::with_capacity(65536, file))), Ok(file) => Ok(Box::new(BufWriter::with_capacity(65536, file))),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::FileCreateDenied, FileTransferErrorType::FileCreateDenied,
format!("{}", err), err.to_string(),
)), )),
} }
} }
@@ -669,7 +669,7 @@ impl FileTransfer for SftpFileTransfer {
Ok(file) => Ok(Box::new(BufReader::with_capacity(65536, file))), Ok(file) => Ok(Box::new(BufReader::with_capacity(65536, file))),
Err(err) => Err(FileTransferError::new_ex( Err(err) => Err(FileTransferError::new_ex(
FileTransferErrorType::NoSuchFileOrDirectory, FileTransferErrorType::NoSuchFileOrDirectory,
format!("{}", err), err.to_string(),
)), )),
} }
} }