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

Wait, file_size in recv is completely useless

Browse Source
This commit is contained in:
ChristianVisintin
2020-12-03 13:43:40 +01:00
parent f8be6b7fac
commit 175acd9f5b
3 changed files with 6 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/filetransfer/mod.rs
View File

@@ -196,5 +196,5 @@ pub trait FileTransfer {
///
/// Receive file from remote with provided name
/// Returns file and its size
fn recv_file(&self, file_name: &Path) -> Result<(Box<dyn Read>, usize), FileTransferError>;
fn recv_file(&self, file_name: &Path) -> Result<Box<dyn Read>, FileTransferError>;
}

14
src/filetransfer/sftp_transfer.rs
View File

@@ -32,7 +32,7 @@ use crate::fs::{FsDirectory, FsEntry, FsFile};
// Includes
use ssh2::{FileStat, Session, Sftp};
use std::io::{Read, Seek, Write};
use std::io::{Read, Write};
use std::net::TcpStream;
use std::path::{Path, PathBuf};
use std::time::{Duration, SystemTime};
@@ -433,7 +433,7 @@ impl FileTransfer for SftpFileTransfer {
/// ### recv_file
///
/// Receive file from remote with provided name
fn recv_file(&self, file_name: &Path) -> Result<(Box<dyn Read>, usize), FileTransferError> {
fn recv_file(&self, file_name: &Path) -> Result<Box<dyn Read>, FileTransferError> {
match self.sftp.as_ref() {
None => Err(FileTransferError::new(FileTransferErrorType::UninitializedSession)),
Some(sftp) => {
@@ -444,15 +444,7 @@ impl FileTransfer for SftpFileTransfer {
};
// Open remote file
match sftp.open(remote_path.as_path()) {
Ok(mut file) => {
let file_size: usize =
file.seek(std::io::SeekFrom::End(0)).unwrap_or(0) as usize;
// rewind
if let Err(err) = file.seek(std::io::SeekFrom::Start(0)) {
return Err(FileTransferError::new(FileTransferErrorType::IoErr(err)));
}
Ok((Box::new(file), file_size))
}
Ok(file) => Ok(Box::new(file)),
Err(err) => Err(FileTransferError::new_ex(FileTransferErrorType::NoSuchFileOrDirectory, format!("{}", err))),
}
}