Tiny code refactor

src/service/build/run_prepare.rs

@@ -87,7 +87,7 @@ pub async fn run(
             let metadata = fs::metadata(&package_source_tar_path).await?;
             let mut permissions = metadata.permissions();
             permissions.set_mode(0o444);
-            fs::set_permissions(package_source_tar_path.clone(), permissions).await?;
+            fs::set_permissions(&package_source_tar_path, permissions).await?;
             let file_name = package_source_tar_path.file_name().unwrap();
             info!("source tar created: {}", file_name.to_string_lossy());
         }

src/service/proxy/package/mod.rs

@@ -56,9 +56,9 @@ pub async fn run(request: Request<PackageRequest>) -> Result<Response<PackageRes
 async fn prepare_source<P: AsRef<Path>>(
     source_name: &str,
     source_hash: &str,
-    source_tar: &P,
+    source_tar: P,
 ) -> Result<(i32, String), anyhow::Error> {
-    let data = read(source_tar).await?;
+    let data = read(&source_tar).await?;
 
     let signature = notary::sign(&data).await?;
 

src/store/archives.rs

@@ -1,6 +1,6 @@
 use anyhow::Result;
 use async_compression::tokio::{bufread::GzipDecoder, write::GzipEncoder};
-use std::path::{Path, PathBuf};
+use std::path::Path;
 use tokio::fs::File;
 use tokio::io::AsyncWriteExt;
 use tokio::io::BufReader;
@@ -50,7 +50,11 @@ where
     Ok(output.into_inner())
 }
 
-pub async fn unpack_tar_gz(target_dir: &PathBuf, source_tar: &Path) -> Result<(), anyhow::Error> {
+pub async fn unpack_tar_gz<P1, P2>(target_dir: P1, source_tar: P2) -> Result<(), anyhow::Error>
+where
+    P1: AsRef<Path>,
+    P2: AsRef<Path>,
+{
     let tar_gz = File::open(source_tar).await?;
     let buf_reader = BufReader::new(tar_gz);
     let gz_decoder = GzipDecoder::new(buf_reader);